Avoid Null Check before g_free(). 82/184882/1
authorRahul Jain <rahul.jain@samsung.com>
Tue, 24 Jul 2018 05:45:10 +0000 (11:15 +0530)
committerRahul Jain <rahul.jain@samsung.com>
Tue, 24 Jul 2018 05:45:56 +0000 (11:15 +0530)
Change-Id: Ibebc50255f075ad0a931fc7bfc180caef1fc14a1
Signed-off-by: Bhaskar Dutta <bhaskar.d@samsung.com>
Signed-off-by: Rahul Jain <rahul.jain@samsung.com>
packaging/net-config.spec
src/signal-handler.c

index 64e1b56..ec96264 100755 (executable)
@@ -1,6 +1,6 @@
 Name:          net-config
 Summary:       TIZEN Network Configuration service
-Version:       1.1.135
+Version:       1.1.136
 Release:       3
 Group:         System/Network
 License:       Apache-2.0
index fc0b1cf..e1cd52e 100755 (executable)
@@ -291,10 +291,8 @@ static void _technology_signal_cb(GDBusConnection *conn,
                        /* Tethering state */
                        wifi_state_set_tech_state(NETCONFIG_WIFI_TECH_TETHERED);
                }
-               if (key)
-                       g_free(key);
-               if (var)
-                       g_variant_unref(var);
+               g_free(key);
+               g_variant_unref(var);
        }
 }
 
@@ -536,8 +534,7 @@ static void _service_signal_cb(GDBusConnection *conn,
                g_free(property);
        }
 done:
-       if (sigvalue)
-               g_free(sigvalue);
+       g_free(sigvalue);
 
        if (variant)
                g_variant_unref(variant);
@@ -563,12 +560,9 @@ static void _dbus_name_changed_cb(GDBusConnection *conn,
 
                connman_register_agent();
        }
-       if (name)
-               g_free(name);
-       if (old)
-               g_free(old);
-       if (new)
-               g_free(new);
+       g_free(name);
+       g_free(old);
+       g_free(new);
 
        return;
 }