Attention is currently required from: Frank Filz.

Lior Suliman would like Frank Filz to review this change.

View Change

uid2grp: Correctly handle getgrouplist() return value.

The POSIX getgrouplist() function returns 0 on success
and an error value on failure, not -1 on failure as it's
commonly assumed.

This change fixes the check for errors after calls to
getgrouplist(), and logs the returned error code to assist
with debugging.

Change-Id: I4a9bb0ad9bd27a08c50383f255851aa40cf611d5
Signed-off-by: Lior Suliman <liorsu@gmail.com>
---
M src/support/uid2grp.c
1 file changed, 11 insertions(+), 11 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/76/1217676/1

To view, visit change 1217676. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I4a9bb0ad9bd27a08c50383f255851aa40cf611d5
Gerrit-Change-Number: 1217676
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu@google.com>
Gerrit-Reviewer: Frank Filz <ffilzlnx@mindspring.com>
Gerrit-Attention: Frank Filz <ffilzlnx@mindspring.com>