diff options
| author | Christian Ambach <ambi@samba.org> | 2011-08-24 16:21:37 +0200 |
|---|---|---|
| committer | Christian Ambach <ambi@samba.org> | 2011-08-24 18:09:11 +0200 |
| commit | fac75f04a8577fd0eb5a3fff9914e4abffa91a1e (patch) | |
| tree | f1c3c613eb5100b56c826f335584c88f9dc59a1c | |
| parent | 6d2f65ba7fe87b45880b80e66407e7a4e49f35a3 (diff) | |
| download | samba-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
| -rw-r--r-- | source3/winbindd/winbindd_group.c | 4 |
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; |