From patchwork Tue Apr 28 19:40:13 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heinrich Schuchardt X-Patchwork-Id: 238775 List-Id: U-Boot discussion From: xypron.glpk at gmx.de (Heinrich Schuchardt) Date: Tue, 28 Apr 2020 21:40:13 +0200 Subject: [PATCH 1/1] cmd/gpt: avoid NULL check before free() Message-ID: <20200428194013.13489-1-xypron.glpk@gmx.de> free() checks if its argument is NULL. Do not duplicate this in the calling code. Signed-off-by: Heinrich Schuchardt Reviewed-by: Tom Rini --- cmd/gpt.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) -- 2.26.2 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; }