From 27486bc5aba2c4a135dd1768b68c077c2cedc63c Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sat, 25 May 2019 01:34:58 +0300 Subject: [PATCH] tests: Fix ap-mgmt-fuzzer exit to free memory This fuzzing wrapper does not go through full hostapd_clean_iface() call to free memory, so call ap_list_deinit() to avoid reporting memory leaks from added AP list entries. This case could not be hit with hostapd/wpa_supplicant since they perform full interface deinit on interface removal. Credit to OSS-Fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=14928 Signed-off-by: Jouni Malinen --- tests/ap-mgmt-fuzzer/ap-mgmt-fuzzer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/ap-mgmt-fuzzer/ap-mgmt-fuzzer.c b/tests/ap-mgmt-fuzzer/ap-mgmt-fuzzer.c index 2adb332e7..9d61b6bcc 100644 --- a/tests/ap-mgmt-fuzzer/ap-mgmt-fuzzer.c +++ b/tests/ap-mgmt-fuzzer/ap-mgmt-fuzzer.c @@ -14,6 +14,7 @@ #include "ap/hw_features.h" #include "ap/ieee802_11.h" #include "ap/sta_info.h" +#include "ap/ap_list.h" const struct wpa_driver_ops *const wpa_drivers[] = @@ -185,6 +186,7 @@ int main(int argc, char *argv[]) ret = 0; fail: hostapd_config_free(ctx.hapd.iconf); + ap_list_deinit(&ctx.iface); eloop_destroy(); os_program_deinit();