Blob Blame History Raw
From 53a6aed98aad73ff51f884bf815bccfa337eb524 Mon Sep 17 00:00:00 2001
From: Barak Sason <bsasonro@redhat.com>
Date: Sun, 18 Aug 2019 17:38:09 +0300
Subject: [PATCH 444/449] mount/fuse - Fixing a coverity issue

Fixed resource leak of dict_value and newkey variables

CID: 1398630

Upstream patch https://review.gluster.org/c/glusterfs/+/23260
> Updates: bz#789278
>
> Change-Id: I589fdc0aecaeb4f446cd29f95bad36ccd7a35beb
> Signed-off-by: Barak Sason <bsasonro@redhat.com>

BUG: 1787310
Change-Id: Id191face7b082e2e8d6e62f60b56248688d396f6
Signed-off-by: Csaba Henk <csaba@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/202760
Tested-by: RHGS Build Bot <nigelb@redhat.com>
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
---
 xlators/mount/fuse/src/fuse-bridge.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/xlators/mount/fuse/src/fuse-bridge.c b/xlators/mount/fuse/src/fuse-bridge.c
index d17320b..f61fa39 100644
--- a/xlators/mount/fuse/src/fuse-bridge.c
+++ b/xlators/mount/fuse/src/fuse-bridge.c
@@ -4165,6 +4165,7 @@ fuse_setxattr(xlator_t *this, fuse_in_header_t *finh, void *msg,
                    "%" PRIu64 ": SETXATTR value allocation failed",
                    finh->unique);
             op_errno = ENOMEM;
+            GF_FREE(newkey);
             goto done;
         }
         memcpy(dict_value, value, fsi->size);
-- 
1.8.3.1