From 5401e9117ce50ecfba3d86b0d4d84ef8fbdc396f Mon Sep 17 00:00:00 2001
From: Monsta <monsta@inbox.ru>
Date: Sun, 16 Nov 2014 17:13:24 +0300
Subject: avoid dereferencing NULL pointer

Closes https://github.com/mate-desktop/mate-applets/pull/114
---
 battstat/acpi-linux.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

(limited to 'battstat/acpi-linux.c')

diff --git a/battstat/acpi-linux.c b/battstat/acpi-linux.c
index 3ac91c29..6f9512ce 100644
--- a/battstat/acpi-linux.c
+++ b/battstat/acpi-linux.c
@@ -350,22 +350,28 @@ gboolean acpi_process_event(struct acpi_info * acpiinfo)
     buffer=g_string_new(NULL);
     g_io_channel_read_line_string   ( acpiinfo->channel,buffer,&i,&gerror);
 
+    gboolean result;
 
     evt = parse_acpi_event(buffer);
       switch (evt) {
         case ACPI_EVENT_AC:
-          return update_ac_info(acpiinfo);
+          result = update_ac_info(acpiinfo);
+          break;
         case ACPI_EVENT_BATTERY_INFO:
           if (update_battery_info(acpiinfo)) {
             /* Update AC info on battery info updates.  This works around
              * a bug in ACPI (as per bug #163013).
              */
-            return update_ac_info(acpiinfo);
+            result = update_ac_info(acpiinfo);
+            break;
           }
           /* fall-through */
         default:
-          return FALSE;
+          result = FALSE;
       }
+
+    g_string_free(buffer, FALSE);
+    return result;
 }
 
 /*
-- 
cgit v1.2.1