aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/tools/perf/tests/api-io.c
diff options
context:
space:
mode:
authorArnaldo Carvalho de Melo <acme@redhat.com>2023-04-12 09:50:08 -0300
committerArnaldo Carvalho de Melo <acme@redhat.com>2023-04-12 10:14:58 -0300
commit9ccbc21166ce149c5f7429cad22f9073d3fab0b1 (patch)
treea63bfc3769a86ba37bfde7875cfb1f23d73f8308 /tools/perf/tests/api-io.c
parentperf bench inject-buildid: Use zfree() to reduce chances of use after free (diff)
downloadwireguard-linux-9ccbc21166ce149c5f7429cad22f9073d3fab0b1.tar.xz
wireguard-linux-9ccbc21166ce149c5f7429cad22f9073d3fab0b1.zip
perf tests api-io: Use zfree() to reduce chances of use after free
Do defensive programming by using zfree() to initialize freed pointers to NULL, so that eventual use after free result in a NULL pointer deref instead of more subtle behaviour. Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/tests/api-io.c')
-rw-r--r--tools/perf/tests/api-io.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/perf/tests/api-io.c b/tools/perf/tests/api-io.c
index 6aea84ca6673..0ba3d5ccebcf 100644
--- a/tools/perf/tests/api-io.c
+++ b/tools/perf/tests/api-io.c
@@ -12,6 +12,7 @@
#include "tests.h"
#include <api/io.h>
#include <linux/kernel.h>
+#include <linux/zalloc.h>
#define TEMPL "/tmp/perf-test-XXXXXX"
@@ -79,7 +80,7 @@ static int setup_test(char path[PATH_MAX], const char *contents,
static void cleanup_test(char path[PATH_MAX], struct io *io)
{
- free(io->buf);
+ zfree(&io->buf);
close(io->fd);
unlink(path);
}