[PATCH 1/1] cmd/gpt: avoid NULL check before free()

free() checks if its argument is NULL. Do not duplicate this in the calling code.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de --- cmd/gpt.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/cmd/gpt.c b/cmd/gpt.c index efaf1bcecb..fb58e4fdc7 100644 --- a/cmd/gpt.c +++ b/cmd/gpt.c @@ -772,11 +772,9 @@ static int do_rename_gpt_parts(struct blk_desc *dev_desc, char *subcomm, out: del_gpt_info(); #ifdef CONFIG_RANDOM_UUID - if (str_disk_guid) - free(str_disk_guid); + free(str_disk_guid); #endif - if (new_partitions) - free(new_partitions); + free(new_partitions); free(partitions_list); return ret; } -- 2.26.2

On Tue, Apr 28, 2020 at 09:40:13PM +0200, Heinrich Schuchardt wrote:
free() checks if its argument is NULL. Do not duplicate this in the calling code.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
Reviewed-by: Tom Rini trini@konsulko.com

On Tue, Apr 28, 2020 at 09:40:13PM +0200, Heinrich Schuchardt wrote:
free() checks if its argument is NULL. Do not duplicate this in the calling code.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de Reviewed-by: Tom Rini trini@konsulko.com
Applied to u-boot/master, thanks!
participants (2)
-
Heinrich Schuchardt
-
Tom Rini