From: Olivier Gayot <olivier.gayot@sigexec.com>
Date: Wed, 20 Jun 2018 09:58:10 +0000 (+0200)
Subject: Fix potential memory leak on Linux
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=445b1925e3d2e239740d7765e5892d0671174e8e;p=i3%2Fi3status

Fix potential memory leak on Linux

The function slurp_all_batteries(), on Linux, allocates memory
dynamically

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>
---

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);