There is a resource leak on fp on an error return path in function parse_symbols that causes a resource leak. Fix this by adding in the missing fclose. Fixes: ef378c3b8233 ("scripts/sorttable: Zero out weak functions in mcount_loc table") Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- scripts/sorttable.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/scripts/sorttable.c b/scripts/sorttable.c index 7b4b3714b1af..e35abf28d037 100644 --- a/scripts/sorttable.c +++ b/scripts/sorttable.c @@ -665,8 +665,10 @@ static int parse_symbols(const char *fname) addr = strtoull(addr_str, NULL, 16); size = strtoull(size_str, NULL, 16); - if (add_field(addr, size) < 0) + if (add_field(addr, size) < 0) { + fclose(fp); return -1; + } } fclose(fp); -- 2.47.2