From b064d501ae68a20f519064662205905dbe9f9b85 Mon Sep 17 00:00:00 2001
From: Hisham Muhammad <hisham@gobolinux.org>
Date: Sun, 18 Feb 2018 10:21:22 -0300
Subject: linux/Battery.c: make sure fd is always closed

Detected by Coverity:
https://scan8.coverity.com/reports.htm#v13252/p10402/fileInstanceId=22093957&defectInstanceId=7543348&mergedDefectId=174180
---
 linux/Battery.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

(limited to 'linux/Battery.c')

diff --git a/linux/Battery.c b/linux/Battery.c
index 34a24011..aedacabc 100644
--- a/linux/Battery.c
+++ b/linux/Battery.c
@@ -110,16 +110,13 @@ static ACPresence procAcpiCheck() {
       char statePath[50];
       xSnprintf((char *) statePath, sizeof statePath, "%s/%s/state", power_supplyPath, entryName);
       FILE* file = fopen(statePath, "r");
-
       if (!file) {
          isOn = AC_ERROR;
          continue;
       }
-
       char* line = String_readLine(file);
-      if (!line) continue;
-
       fclose(file);
+      if (!line) continue;
 
       const char *isOnline = String_getToken(line, 2);
       free(line);
-- 
cgit v1.2.3