From: Hyunwoo Park <dshw.park@samsung.com>
Date: Thu, 30 May 2024 02:15:05 +0000 (+0000)
Subject: iolog: fix Null pointer dereferences (FORWARD_NULL)
X-Git-Tag: fio-3.38~65^2~1
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=a70e681c58b0da219dd57749df6e7c97607b29bf;p=fio.git

iolog: fix Null pointer dereferences (FORWARD_NULL)

CID 494150: Null pointer dereferences (FORWARD_NULL) @ iolog.c:148 in ipo_special()
This patch removes the possibility of null pointer dereferencing(io_u->file)
throughout the call stack of get_io_u() → read_iolog_get() → dp_fill_dspec_data()

Signed-off-by: Hyunwoo Park <dshw.park@samsung.com>
---

diff --git a/iolog.c b/iolog.c
index 37ad0d2a..e3a56db9 100644
--- a/iolog.c
+++ b/iolog.c
@@ -236,15 +236,15 @@ int read_iolog_get(struct thread_data *td, struct io_u *io_u)
 						io_u->buflen, io_u->file->file_name);
 			if (ipo->delay)
 				iolog_delay(td, ipo->delay);
+
+			if (td->o.dp_type != FIO_DP_NONE)
+				dp_fill_dspec_data(td, io_u);
 		} else {
 			elapsed = mtime_since_genesis();
 			if (ipo->delay > elapsed)
 				usec_sleep(td, (ipo->delay - elapsed) * 1000);
 		}
 
-		if (td->o.dp_type != FIO_DP_NONE)
-			dp_fill_dspec_data(td, io_u);
-
 		free(ipo);
 
 		if (io_u->ddir != DDIR_WAIT)