diff options
| author | Dmitry V. Levin <ldv@altlinux.org> | 2021-05-12 15:00:00 +0000 |
|---|---|---|
| committer | Dmitry V. Levin <ldv@altlinux.org> | 2021-05-12 15:00:00 +0000 |
| commit | 806c1f7a0d08ae2be40b39773ddf16ea5c8a1128 (patch) | |
| tree | e2b9e224ed2a352dd6df15605327ed06a4091c7e | |
| parent | 43ac25c2de7ba21fb8f9e4d4db0c01c1408da0b8 (diff) | |
| download | elfutils-806c1f7a0d08ae2be40b39773ddf16ea5c8a1128.tar.gz | |
elfcompress: remove redundant assignment
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.")
| -rw-r--r-- | src/ChangeLog | 3 | ||||
| -rw-r--r-- | src/elfcompress.c | 1 |
2 files changed, 3 insertions, 1 deletions
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; } |