Re: [PATCH 2/2] elfcompress: remove redundant assignment

2021-05-12 Thread Mark Wielaard
Hi Dmitry, On Wed, May 12, 2021 at 03:00:00PM +, Dmitry V. Levin wrote: > 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: e

[PATCH 2/2] elfcompress: remove redundant assignment

2021-05-12 Thread Dmitry V. Levin
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.")