elfcompress: fix exit status regression in case of "Nothing to do"

When elfcompress decides that no section data needs to be updated and
therefore the file does not have to be rewritten, it still has to exit
with a zero status indicating success.

Resolves: https://sourceware.org/bugzilla/show_bug.cgi?id=27856
Fixes: c497478390de ("elfcompress: Replace cleanup() with label")

diff --git a/src/ChangeLog b/src/ChangeLog
index c5ecc05..47c7eb7 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2021-05-12  Dmitry V. Levin  <ldv@altlinux.org>
+
+       * elfcompress.c (process_file): Set res to 0 in case of "Nothing to do".
+
 2021-04-19  Martin Liska  <mliska@suse.cz>
 
        * elfclassify.c (run_classify): Use startswith.

diff --git a/src/elfcompress.c b/src/elfcompress.c
index d5bc330..7340e87 100644 (file)
--- a/src/elfcompress.c
+++ b/src/elfcompress.c
@@ -522,6 +522,7 @@ process_file (const char *fname)
       if (verbose > 0)
        printf ("Nothing to do.\n");
       fnew = NULL;
+      res = 0;
       goto cleanup;
     }