devilhorns pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=194f1c77b518494d52fc331ca668298101aa1188

commit 194f1c77b518494d52fc331ca668298101aa1188
Author: Christopher Michael <[email protected]>
Date:   Tue Apr 28 11:31:24 2020 -0400

    e_system_main: Fix resource leak
    
    Coverity reports a leak here because we are not closing the previously
    opened file. Fix that.
    
    fixes CID1427985
---
 src/bin/system/e_system_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bin/system/e_system_main.c b/src/bin/system/e_system_main.c
index 289435fa2..570adc70a 100644
--- a/src/bin/system/e_system_main.c
+++ b/src/bin/system/e_system_main.c
@@ -38,7 +38,7 @@ _etc_enlightenment_system_conf_check(const char *sys)
    if (gn < 0)
      {
         ERR("User %i member of too many groups\n", uid);
-        return 0;
+        goto deny;
      }
    while (fgets(buf, sizeof(buf), f))
      {

-- 


Reply via email to