Hi,
On Thu, 2015-09-03 at 18:41 +0530, Harish Jenny K N wrote:
> Moved the supplicant debug in bss_property after NULL
> check for const char *key variable.
>
> ---
> gsupplicant/supplicant.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
> index 98ca94b..5d3be8b 100644
> --- a/gsupplicant/supplicant.c
> +++ b/gsupplicant/supplicant.c
> @@ -1738,11 +1738,11 @@ static void bss_property(const char *key,
> DBusMessageIter *iter,
> if (!bss->interface)
> return;
>
> - SUPPLICANT_DBG("key %s", key);
> -
> if (!key)
> return;
>
> + SUPPLICANT_DBG("key %s", key);
> +
> if (g_strcmp0(key, "BSSID") == 0) {
> DBusMessageIter array;
> unsigned char *addr;
This merely prints out "key (null)" without further side effects. It's a
useful printout when there is something fishy going on.
Cheers,
Patrik
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
