Joursoir has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/64351 )
Change subject: cli_classic: fix memory leak ......................................................................
cli_classic: fix memory leak
If parameter --wp-region is used and wp_region is allocated, free the last one at the exit of the application.
Found-by: scan-build, clang v13.0.1 Signed-off-by: Alexander Goncharov chat@joursoir.net Change-Id: I8520e302e9d63ed1215c5d9beb90a93fb52a91fe --- M cli_classic.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/51/64351/1
diff --git a/cli_classic.c b/cli_classic.c index 9543009..5cd05a7 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -1100,6 +1100,8 @@ free(referencefile); free(layoutfile); free(pparam); + if (set_wp_region && wp_region) + free(wp_region); /* clean up global variables */ free((char *)chip_to_probe); /* Silence! Freeing is not modifying contents. */ chip_to_probe = NULL;