dpkg: run_package_script() returns 0 if all ok and non-zero if failure.

The result code was checked incorrectly in two places.
(from "Kim B. Heino" <Kim.Heino@bluegiga.com>)


git-svn-id: svn://busybox.net/trunk/busybox@18278 69ca8d6d-28ef-0310-b511-8ec308f3f277

1 files changed, 3 insertions, 5 deletions

diff --git a/archival/dpkg.c b/archival/dpkg.c
index 0a42deb39..c64410096 100644
--- a/archival/dpkg.c
+++ b/archival/dpkg.c
@@ -1306,14 +1306,12 @@ static void remove_package(const unsigned package_num, int noisy)
         char **exclude_files;
         char list_name[package_name_length + 25];
         char conffile_name[package_name_length + 30];
-        int return_value;
 
         if (noisy)
                 printf("Removing %s (%s)...\n", package_name, package_version);
 
         /* run prerm script */
-        return_value = run_package_script(package_name, "prerm");
-        if (return_value == -1) {
+        if (run_package_script(package_name, "prerm") != 0) {
                 bb_error_msg_and_die("script failed, prerm failure");
         }
 
@@ -1381,8 +1379,8 @@ static void purge_package(const unsigned package_num)
         free(exclude_files);
 
         /* run postrm script */
-        if (run_package_script(package_name, "postrm") == -1) {
-                bb_error_msg_and_die("postrm fialure.. set status to what?");
+        if (run_package_script(package_name, "postrm") != 0) {
+                bb_error_msg_and_die("postrm failure.. set status to what?");
         }
 
         /* Change package status */