[PATCH 2/2] elfcompress: remove redundant assignment

Dmitry V. Levin ldv@altlinux.org
Wed May 12 15:00:00 GMT 2021


At the point of "Nothing to do" fnew variable has not been assigned
after initialization, so it does not have to be reset to NULL.

Note that any reset of fnew to NULL has to preceded with free(fnew).

Fixes: ed62996defc6 ("elfcompress: Don't rewrite file if no section data needs to be updated.")
---
 src/ChangeLog     | 3 +++
 src/elfcompress.c | 1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/ChangeLog b/src/ChangeLog
index 47c7eb79..de130f79 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,8 @@
 2021-05-12  Dmitry V. Levin  <ldv@altlinux.org>
 
+	* elfcompress.c (process_file): Remove redundant assignment in case of
+	"Nothing to do".
+
 	* elfcompress.c (process_file): Set res to 0 in case of "Nothing to do".
 
 2021-04-19  Martin Liska  <mliska@suse.cz>
diff --git a/src/elfcompress.c b/src/elfcompress.c
index 7340e878..05f3bc2c 100644
--- a/src/elfcompress.c
+++ b/src/elfcompress.c
@@ -521,7 +521,6 @@ process_file (const char *fname)
     {
       if (verbose > 0)
 	printf ("Nothing to do.\n");
-      fnew = NULL;
       res = 0;
       goto cleanup;
     }
-- 
ldv


More information about the Elfutils-devel mailing list