From: Denis Pronin <dannftk@yandex.ru>
Date: Thu, 10 Mar 2022 10:34:53 +0000 (+0300)
Subject: improvements in dup_files function
X-Git-Tag: fio-3.30~15^2
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=5d7de92eaff9ac2514e5992320c273c82d5d282d;p=fio.git

improvements in dup_files function

cleared allocation of td->files when duplicating files,
call 'assert', before iterating over original thread_data,
that prevents possible segmentation fault when duplicating files

Signed-off-by: Denis Pronin <dannftk@yandex.ru>
---

diff --git a/filesetup.c b/filesetup.c
index 7c32d0af..ab6c488b 100644
--- a/filesetup.c
+++ b/filesetup.c
@@ -2031,11 +2031,12 @@ void dup_files(struct thread_data *td, struct thread_data *org)
 	if (!org->files)
 		return;
 
-	td->files = malloc(org->files_index * sizeof(f));
+	td->files = calloc(org->files_index, sizeof(f));
 
 	if (td->o.file_lock_mode != FILE_LOCK_NONE)
 		td->file_locks = malloc(org->files_index);
 
+	assert(org->files_index >= org->o.nr_files);
 	for_each_file(org, f, i) {
 		struct fio_file *__f;