From: Marcel Holtmann <marcel@holtmann.org>
Date: Mon, 23 Nov 2009 23:27:22 +0000 (+0100)
Subject: Get rid of the pointless checks before g_free in GPRS engine
X-Git-Tag: 0.11~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=231aca4a69418fad4434d7e5902d23161effca68;p=platform%2Fupstream%2Fofono.git

Get rid of the pointless checks before g_free in GPRS engine
---

diff --git a/src/gprs.c b/src/gprs.c
index 5df3619..5f2959f 100644
--- a/src/gprs.c
+++ b/src/gprs.c
@@ -159,20 +159,11 @@ static struct pri_context *gprs_context_by_path(struct ofono_gprs *gprs,
 
 static void context_settings_free(struct context_settings *settings)
 {
-	if (settings->interface)
-		g_free(settings->interface);
-
-	if (settings->ip)
-		g_free(settings->ip);
-
-	if (settings->netmask)
-		g_free(settings->netmask);
-
-	if (settings->gateway)
-		g_free(settings->gateway);
-
-	if (settings->dns)
-		g_strfreev(settings->dns);
+	g_free(settings->interface);
+	g_free(settings->ip);
+	g_free(settings->netmask);
+	g_free(settings->gateway);
+	g_strfreev(settings->dns);
 
 	g_free(settings);
 }
@@ -760,8 +751,7 @@ static void pri_context_destroy(gpointer userdata)
 		ctx->settings = NULL;
 	}
 
-	if (ctx->path)
-		g_free(ctx->path);
+	g_free(ctx->path);
 
 	g_free(ctx);
 }
@@ -1723,20 +1713,11 @@ static gboolean load_context(struct ofono_gprs *gprs, const char *group)
 	ret = TRUE;
 
 err:
-	if (name)
-		g_free(name);
-
-	if (typestr)
-		g_free(typestr);
-
-	if (username)
-		g_free(username);
-
-	if (password)
-		g_free(password);
-
-	if (apn)
-		g_free(apn);
+	g_free(name);
+	g_free(typestr);
+	g_free(username);
+	g_free(password);
+	g_free(apn);
 
 	return ret;
 }