From 2b07ca9a10bc1e942eb39c355597d8cce098cec9 Mon Sep 17 00:00:00 2001 From: Hyunwoo Park Date: Thu, 30 May 2024 02:15:05 +0000 Subject: [PATCH] iolog: fix Coverity Scan defect MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- iolog.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/iolog.c b/iolog.c index 37ad0d2a16..e3a56db957 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)