From: Daniel Wagner <[email protected]>
The NULL check is unnecessary, because g_supplicant_interface_scan()
provides a valid pointer. If data were to be NULL, there would have
been a crash already because we already derefence it a few lines
above.
---
gsupplicant/supplicant.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
index 649e514..eaea947 100644
--- a/gsupplicant/supplicant.c
+++ b/gsupplicant/supplicant.c
@@ -2798,7 +2798,7 @@ static void interface_scan_result(const char *error,
data->interface->scan_data = data->user_data;
}
- if (data && data->scan_params)
+ if (data->scan_params)
g_supplicant_free_scan_params(data->scan_params);
dbus_free(data);
--
1.8.4.474.g128a96c
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
