emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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

Browse Source
This commit is contained in:
Denis Vlasenko 2007-03-19 21:48:56 +00:00
parent 97fd6d81b4
commit 88a2aa98e0
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
archival/dpkg.c
View File

@ -643,7 +643,6 @@ static unsigned fill_package_struct(char *control_buffer)
return -1; return -1;
} }
num = search_package_hashtable(new_node->name, new_node->version, VER_EQUAL); 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; package_hashtable[num] = new_node;
return num; return num;
@ -1739,10 +1738,8 @@ int dpkg_main(int argc, char **argv)
} }
for (i = 0; i < PACKAGE_HASH_PRIME; i++) { 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++) { for (i = 0; i < STATUS_HASH_PRIME; i++) {
free(status_hashtable[i]); free(status_hashtable[i]);