* free() checks if its argument is NULL. Remove duplicate checks. * Remove duplicate free(ovcopy).
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com --- boot/image-fit.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/boot/image-fit.c b/boot/image-fit.c index f01cafe4e2..401cdd3b55 100644 --- a/boot/image-fit.c +++ b/boot/image-fit.c @@ -1885,8 +1885,7 @@ int fit_conf_get_node(const void *fit, const char *conf_uname) conf_uname, fdt_strerror(noffset)); }
- if (conf_uname_copy) - free(conf_uname_copy); + free(conf_uname_copy);
return noffset; } @@ -2420,9 +2419,6 @@ int boot_get_fdt_fit(bootm_headers_t *images, ulong addr, } fdt_pack(base); len = fdt_totalsize(base); - - free(ovcopy); - ovcopy = NULL; } #else printf("config with overlays but CONFIG_OF_LIBFDT_OVERLAY not set\n"); @@ -2440,11 +2436,9 @@ out: *fit_uname_configp = fit_uname_config;
#ifdef CONFIG_OF_LIBFDT_OVERLAY - if (ovcopy) - free(ovcopy); + free(ovcopy); #endif - if (fit_uname_config_copy) - free(fit_uname_config_copy); + free(fit_uname_config_copy); return fdt_noffset; } #endif