summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorChristian Ambach <ambi@samba.org>2011-08-24 16:21:37 +0200
committerChristian Ambach <ambi@samba.org>2011-08-24 18:09:11 +0200
commitfac75f04a8577fd0eb5a3fff9914e4abffa91a1e (patch)
treef1c3c613eb5100b56c826f335584c88f9dc59a1c /source3
parent6d2f65ba7fe87b45880b80e66407e7a4e49f35a3 (diff)
downloadsamba-fac75f04a8577fd0eb5a3fff9914e4abffa91a1e.tar.gz
samba-fac75f04a8577fd0eb5a3fff9914e4abffa91a1e.tar.bz2
samba-fac75f04a8577fd0eb5a3fff9914e4abffa91a1e.zip
s3:winbindd fix a return code check
talloc_traverse_dict will return with -1 in case of an error and might return positive values that indicate the count of found entries Autobuild-User: Christian Ambach <ambi@samba.org> Autobuild-Date: Wed Aug 24 18:09:11 CEST 2011 on sn-devel-104
Diffstat (limited to 'source3')
-rw-r--r--source3/winbindd/winbindd_group.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/winbindd/winbindd_group.c b/source3/winbindd/winbindd_group.c
index 1e4ad5fa8a..2f8ba6a023 100644
--- a/source3/winbindd/winbindd_group.c
+++ b/source3/winbindd/winbindd_group.c
@@ -115,7 +115,7 @@ NTSTATUS winbindd_print_groupmembers(struct talloc_dict *members,
c.len = 0;
res = talloc_dict_traverse(members, getgr_calc_memberlen, &c);
- if (res != 0) {
+ if (res == -1) {
DEBUG(5, ("talloc_dict_traverse failed\n"));
return NT_STATUS_INTERNAL_ERROR;
}
@@ -128,7 +128,7 @@ NTSTATUS winbindd_print_groupmembers(struct talloc_dict *members,
}
res = talloc_dict_traverse(members, getgr_unparse_members, &m);
- if (res != 0) {
+ if (res == -1) {
DEBUG(5, ("talloc_dict_traverse failed\n"));
TALLOC_FREE(m.buf);
return NT_STATUS_INTERNAL_ERROR;