From: Daniel Wagner <[email protected]>
The NULL check is unnecessary, because the caller 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 23ea41a..801a181 100644
--- a/gsupplicant/supplicant.c
+++ b/gsupplicant/supplicant.c
@@ -2999,7 +2999,7 @@ static void interface_autoscan_result(const char *error,
g_free(data->path);
- if (data && data->callback)
+ if (data->callback)
data->callback(err, data->interface, data->user_data);
dbus_free(data);
--
1.8.4.474.g128a96c
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
