From 445b1925e3d2e239740d7765e5892d0671174e8e Mon Sep 17 00:00:00 2001 From: Olivier Gayot Date: Wed, 20 Jun 2018 11:58:10 +0200 Subject: [PATCH] Fix potential memory leak on Linux The function slurp_all_batteries(), on Linux, allocates memory dynamically Signed-off-by: Olivier Gayot --- src/print_battery_info.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/print_battery_info.c b/src/print_battery_info.c index 7a462f7..72c2d6f 100644 --- a/src/print_battery_info.c +++ b/src/print_battery_info.c @@ -450,8 +450,11 @@ static bool slurp_all_batteries(struct battery_info *batt_info, yajl_gen json_ge .present_rate = 0, .status = CS_UNKNOWN, }; - if (!slurp_battery_info(&batt_buf, json_gen, buffer, i, globbuf.gl_pathv[i], format_down)) + if (!slurp_battery_info(&batt_buf, json_gen, buffer, i, globbuf.gl_pathv[i], format_down)) { + globfree(&globbuf); + free(globpath); return false; + } is_found = true; add_battery_info(batt_info, &batt_buf); -- 2.39.5