>From Niels de Vos <nde...@redhat.com>:

Niels de Vos has uploaded a new change for review.

  https://review.gerrithub.io/265307

Change subject: FSAL_GLUSTER: bail out if there is no ACL-entry
......................................................................

FSAL_GLUSTER: bail out if there is no ACL-entry

Coverity reported:

    Unused value (UNUSED_VALUE)
    returned_value: Assigning value from acl_get_entry(dup_acl, 1,
                    &d_entry) to ret here, but that stored value is
                    overwritten before it can be used.

When getting the ACL-entry fails, the structure will not contain any
valid data and we should not try to get the permission set from it.

Change-Id: I2793733861c2fa7b56edf61262dab4319985a9ae
CID: 131057
Signed-off-by: Niels de Vos <nde...@redhat.com>
---
M src/FSAL/FSAL_GLUSTER/posix_acls.c
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/07/265307/1
-- 
To view, visit https://review.gerrithub.io/265307
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2793733861c2fa7b56edf61262dab4319985a9ae
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Niels de Vos <nde...@redhat.com>

------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://makebettercode.com/inteldaal-eval
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to