backend: Removing memory leak in run_threads()
authorErwan Velu <erwan@redhat.com>
Tue, 3 Oct 2017 09:19:57 +0000 (11:19 +0200)
committerJens Axboe <axboe@kernel.dk>
Tue, 3 Oct 2017 15:06:33 +0000 (09:06 -0600)
The fd structure must be freed before breaking the foreach() unless it
leaks its memory

This patch add a free() call in the last break statement.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
backend.c

index 029d0054d01ded07fab0aa5fecfac3436eb33973..b1995ef4ff813de82e8e6b2045b78b709ce8e29c 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -2346,6 +2346,7 @@ reap:
                                fio_terminate_threads(TERMINATE_ALL);
                                fio_abort = 1;
                                nr_started--;
                                fio_terminate_threads(TERMINATE_ALL);
                                fio_abort = 1;
                                nr_started--;
+                               free(fd);
                                break;
                        }
                        dprint(FD_MUTEX, "done waiting on startup_mutex\n");
                                break;
                        }
                        dprint(FD_MUTEX, "done waiting on startup_mutex\n");