From: Tomohiro Kusumi <tkusumi@xxxxxxxxxx>
This is unlikely to happen in the first place since ->filename_format
has a default value, and option parser can also detect an empty string,
but it should return buf which is sprintf'd right before returning.
A caller expects this function to return filename string (which is
buf arg itself in this case), and it also doesn't handle NULL return.
Signed-off-by: Tomohiro Kusumi <tkusumi@xxxxxxxxxx>
---
init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/init.c b/init.c
index 4a72255..6af6c45 100644
--- a/init.c
+++ b/init.c
@@ -1121,7 +1121,7 @@ static char *make_filename(char *buf, size_t buf_size,struct thread_options *o,
if (!o->filename_format || !strlen(o->filename_format)) {
sprintf(buf, "%s.%d.%d", jobname, jobnum, filenum);
- return NULL;
+ return buf;
}
for (f = &fpre_keywords[0]; f->keyword; f++)
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
