[PATCH] wpa_supplicant: Fix misplaced os_free

Jouni Malinen j
Mon Feb 3 09:37:23 PST 2014


On Mon, Feb 03, 2014 at 05:49:02PM +0530, Prashanth Kumar wrote:
> os_free has wfd_dev_info_hex as an argument which is defined within
> CONFIG_NO_STDOUT_DEBUG

Thanks, applied.

> diff --git a/wpa_supplicant/p2p_supplicant.c b/wpa_supplicant/p2p_supplicant.c
> @@ -1684,9 +1684,12 @@ static void wpas_dev_found(void *ctx, const u8 *addr,
> -#endif /* CONFIG_NO_STDOUT_DEBUG */
>  
> +#ifdef CONFIG_WIFI_DISPLAY
>  	os_free(wfd_dev_info_hex);
> +#endif /* CONFIG_WIFI_DISPLAY */
> +
> +#endif /* CONFIG_NO_STDOUT_DEBUG */

Though, I skipped the unnecessary #ifdef CONFIG_WIFI_DISPLAY to keep
this a bit nicer to look at.
 
-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list