From: Denis Vlasenko <vda.linux@googlemail.com>
Date: Mon, 19 Mar 2007 21:48:56 +0000 (-0000)
Subject: dpkg: free_package(NULL) is ok, so don't test for NULL, just free.
X-Git-Tag: 1_5_1~18
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=88a2aa98e06cf6b0aa00a3cf88376a1d3497fb2c;p=oweals%2Fbusybox.git

dpkg: free_package(NULL) is ok, so don't test for NULL, just free.
---

diff --git a/archival/dpkg.c b/archival/dpkg.c
index ebdf6c68a..0a42deb39 100644
--- a/archival/dpkg.c
+++ b/archival/dpkg.c
@@ -643,8 +643,7 @@ static unsigned fill_package_struct(char *control_buffer)
 		return -1;
 	}
 	num = search_package_hashtable(new_node->name, new_node->version, VER_EQUAL);
-	if (package_hashtable[num] != NULL)
-		free_package(package_hashtable[num]);
+	free_package(package_hashtable[num]);
 	package_hashtable[num] = new_node;
 	return num;
 }
@@ -1739,9 +1738,7 @@ int dpkg_main(int argc, char **argv)
 		}
 
 		for (i = 0; i < PACKAGE_HASH_PRIME; i++) {
-			if (package_hashtable[i] != NULL) {
-				free_package(package_hashtable[i]);
-			}
+			free_package(package_hashtable[i]);
 		}
 
 		for (i = 0; i < STATUS_HASH_PRIME; i++) {