[PATCH libICE 01/13] free() can handle NULL so remove the check

walter harms wharms at bfs.de
Wed Oct 18 15:47:48 UTC 2017


>From f6d630eda04e02713da059519f56dd2dd75a703c Mon Sep 17 00:00:00 2001


free() can handle NULL so remove the check

---
 src/authutil.c | 24 ++++++++++++------------
 src/misc.c     |  3 +--
 src/process.c  | 41 +++++++++++++++--------------------------
 src/shutdown.c | 44 +++++++++++---------------------------------
 4 files changed, 39 insertions(+), 73 deletions(-)

diff --git a/src/authutil.c b/src/authutil.c
index ca0504a..d7bcef9 100644
--- a/src/authutil.c
+++ b/src/authutil.c
@@ -111,8 +111,8 @@ IceAuthFileName (void)

     if (size > bsize)
     {
-	if (buf)
-	    free (buf);
+
+	free (buf);
 	buf = malloc (size);
 	if (!buf) {
 	    bsize = 0;
@@ -266,11 +266,11 @@ IceReadAuthFileEntry (

  bad:

-    if (local.protocol_name) free (local.protocol_name);
-    if (local.protocol_data) free (local.protocol_data);
-    if (local.network_id) free (local.network_id);
-    if (local.auth_name) free (local.auth_name);
-    if (local.auth_data) free (local.auth_data);
+    free (local.protocol_name);
+    free (local.protocol_data);
+    free (local.network_id);
+    free (local.auth_name);
+    free (local.auth_data);

     return (NULL);
 }
@@ -284,11 +284,11 @@ IceFreeAuthFileEntry (
 {
     if (auth)
     {
-	if (auth->protocol_name) free (auth->protocol_name);
-	if (auth->protocol_data) free (auth->protocol_data);
-	if (auth->network_id) free (auth->network_id);
-	if (auth->auth_name) free (auth->auth_name);
-	if (auth->auth_data) free (auth->auth_data);
+	free (auth->protocol_name);
+	free (auth->protocol_data);
+	free (auth->network_id);
+	free (auth->auth_name);
+	free (auth->auth_data);
 	free (auth);
     }
 }
diff --git a/src/misc.c b/src/misc.c
index d2e9150..87d6335 100644
--- a/src/misc.c
+++ b/src/misc.c
@@ -54,8 +54,7 @@ IceAllocScratch (
 {
     if (!iceConn->scratch || size > iceConn->scratch_size)
     {
-	if (iceConn->scratch)
-	    free (iceConn->scratch);
+	free (iceConn->scratch);

 	iceConn->scratch = malloc (size);
 	iceConn->scratch_size = size;
diff --git a/src/process.c b/src/process.c
index 4100a83..a9a8d08 100644
--- a/src/process.c
+++ b/src/process.c
@@ -1026,8 +1026,7 @@ ProcessConnectionSetup (
 		iceConn->connection_status = IceConnectRejected;
 	    }

-	    if (hostname)
-		free (hostname);
+	    free (hostname);
 	}

 	if (iceConn->connection_status == IceConnectRejected)
@@ -1080,8 +1079,7 @@ ProcessConnectionSetup (
 	if (authData && authDataLen > 0)
 	    free (authData);

-	if (errorString)
-	    free (errorString);
+	free (errorString);
     }

     if (accept_setup_now)
@@ -1369,8 +1367,7 @@ ProcessAuthReply (
 		    status = IcePaAuthAccepted;
 		}

-		if (hostname)
-		    free (hostname);
+		free (hostname);
 	    }

 	    if (status != IcePaAuthAccepted)
@@ -1444,8 +1441,7 @@ ProcessAuthReply (
 		    status = IcePaAuthAccepted;
 		}

-		if (hostname)
-		    free (hostname);
+		free (hostname);
 	    }

 	    if (status == IcePaAuthRejected)
@@ -1559,18 +1555,15 @@ ProcessAuthReply (
 		_IceErrorSetupFailed (iceConn, ICE_ProtocolSetup,
 		    failureReason);

-		if (failureReason)
-		    free (failureReason);
+		free (failureReason);
 	    }
 	}


 	if (free_setup_info)
 	{
-	    if (iceConn->protosetup_to_me->his_vendor)
-		free (iceConn->protosetup_to_me->his_vendor);
-	    if (iceConn->protosetup_to_me->his_release)
-		free (iceConn->protosetup_to_me->his_release);
+	    free (iceConn->protosetup_to_me->his_vendor);
+	    free (iceConn->protosetup_to_me->his_release);
 	    free (iceConn->protosetup_to_me);
 	    iceConn->protosetup_to_me = NULL;
 	}
@@ -1587,8 +1580,8 @@ ProcessAuthReply (
     if (authData && authDataLen > 0)
 	free (authData);

-    if (errorString)
-	free (errorString);
+
+    free (errorString);

     IceDisposeCompleteMessage (iceConn, replyData);
     return (0);
@@ -2071,8 +2064,7 @@ ProcessProtocolSetup (
 	            ICE_ProtocolSetup, "None of the authentication protocols specified are supported and host-based authentication failed");
 	    }

-	    if (hostname)
-		free (hostname);
+	    free (hostname);
 	}
     }
     else
@@ -2118,8 +2110,8 @@ ProcessProtocolSetup (
 	if (authData && authDataLen > 0)
 	    free (authData);

-	if (errorString)
-	    free (errorString);
+
+	free (errorString);
     }

     if (accept_setup_now)
@@ -2202,16 +2194,13 @@ ProcessProtocolSetup (

 	    _IceErrorSetupFailed (iceConn, ICE_ProtocolSetup, failureReason);

-	    if (failureReason)
-		free (failureReason);
+	    free (failureReason);
 	}
     }

-    if (vendor)
-	free (vendor);

-    if (release)
-	free (release);
+    free (vendor);
+    free (release);

     if (hisAuthCount > 0)
     {
diff --git a/src/shutdown.c b/src/shutdown.c
index 5def0b4..90e9ded 100644
--- a/src/shutdown.c
+++ b/src/shutdown.c
@@ -282,39 +282,17 @@ _IceFreeConnection (
     if (iceConn->trans_conn)
 	_IceTransClose (iceConn->trans_conn);

-    if (iceConn->connection_string)
-	free (iceConn->connection_string);
-
-    if (iceConn->vendor)
-	free (iceConn->vendor);
-
-    if (iceConn->release)
-	free (iceConn->release);
-
-    if (iceConn->inbuf)
-	free (iceConn->inbuf);
-
-    if (iceConn->outbuf)
-	free (iceConn->outbuf);
-
-    if (iceConn->scratch)
-	free (iceConn->scratch);
-
-    if (iceConn->process_msg_info)
-	free (iceConn->process_msg_info);
-
-    if (iceConn->connect_to_you)
-	free (iceConn->connect_to_you);
-
-    if (iceConn->protosetup_to_you)
-	free (iceConn->protosetup_to_you);
-
-    if (iceConn->connect_to_me)
-	free (iceConn->connect_to_me);
-
-    if (iceConn->protosetup_to_me)
-	free (iceConn->protosetup_to_me);
-
+    free (iceConn->connection_string);
+    free (iceConn->vendor);
+    free (iceConn->release);
+    free (iceConn->inbuf);
+    free (iceConn->outbuf);
+    free (iceConn->scratch);
+    free (iceConn->process_msg_info);
+    free (iceConn->connect_to_you);
+    free (iceConn->protosetup_to_you);
+    free (iceConn->connect_to_me);
+    free (iceConn->protosetup_to_me);
     free (iceConn);
 }

-- 
2.1.4



More information about the xorg-devel mailing list