From: Jouni Malinen <jouni@codeaurora.org>
Date: Fri, 24 May 2019 22:34:58 +0000 (+0300)
Subject: tests: Fix ap-mgmt-fuzzer exit to free memory
X-Git-Tag: hostap_2_9~271
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=27486bc5aba2c4a135dd1768b68c077c2cedc63c;p=thirdparty%2Fhostap.git

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 <jouni@codeaurora.org>
---

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();