From 7b61434257f68749bfceee481fa505de7f945747 Mon Sep 17 00:00:00 2001
From: Saurabh Badhwar <sbsaurabhbadhwar9@gmail.com>
Date: Fri, 13 Jan 2017 12:59:51 +0530
Subject: fuse: Fix a possible resource leak under GF_SOLARIS_HOST_OS in
 fuse-helpers.c

Change-Id: Ie367a6dec2a0d5848631b19ebbe39ceafa954a60
BUG: 1412918
Signed-off-by: Saurabh Badhwar <sbsaurabhbadhwar9@gmail.com>
Reviewed-on: http://review.gluster.org/16395
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Saravanakumar Arumugam <sarumuga@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
---
 xlators/mount/fuse/src/fuse-helpers.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/xlators/mount/fuse/src/fuse-helpers.c b/xlators/mount/fuse/src/fuse-helpers.c
index 3e541979dc7..b65e6ebfc7b 100644
--- a/xlators/mount/fuse/src/fuse-helpers.c
+++ b/xlators/mount/fuse/src/fuse-helpers.c
@@ -249,8 +249,10 @@ out:
                                 ngrps = MIN(prcred->pr_ngroups,
                                             FUSE_MAX_AUX_GROUPS);
                                 if (call_stack_alloc_groups (frame->root,
-                                                             ngrps) != 0)
+                                                             ngrps) != 0) {
+                                        fclose (fp);
                                         return;
+                                }
                         }
                         fclose (fp);
                  }
-- 
cgit