From db54100ea3382d64ba739584b1458f161c10d693 Mon Sep 17 00:00:00 2001 From: Gerald Carter Date: Wed, 17 Aug 2005 13:34:23 +0000 Subject: r9361: patch from Ed Plese to fix a faulty error in winbindd caused by users with no supplementary groups. --- WHATSNEW.txt | 5 +++++ source/nsswitch/winbindd_ads.c | 25 ++++++++++++------------- 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/WHATSNEW.txt b/WHATSNEW.txt index f202187a932..eb6f0eccda8 100644 --- a/WHATSNEW.txt +++ b/WHATSNEW.txt @@ -93,6 +93,11 @@ o James Peach * BUG 2314: Fix const compiler warnings in the quota code. +o Ed Plese + * Fix faulty logic which caused winbindd to return failure + when a user passessed no supplementary groups. + + Release Notes for older release follow: diff --git a/source/nsswitch/winbindd_ads.c b/source/nsswitch/winbindd_ads.c index 50899714475..a7849e59b16 100644 --- a/source/nsswitch/winbindd_ads.c +++ b/source/nsswitch/winbindd_ads.c @@ -499,28 +499,27 @@ static NTSTATUS lookup_usergroups_alt(struct winbindd_domain *domain, } count = ads_count_replies(ads, res); - if (count == 0) { - DEBUG(5,("lookup_usergroups: No supp groups found\n")); - - status = ads_ntstatus(rc); - goto done; - } *user_sids = NULL; *num_groups = 0; + /* always add the primary group to the sid array */ add_sid_to_array(mem_ctx, primary_group, user_sids, num_groups); - for (msg = ads_first_entry(ads, res); msg; - msg = ads_next_entry(ads, msg)) { - DOM_SID group_sid; + if (count > 0) { + for (msg = ads_first_entry(ads, res); msg; + msg = ads_next_entry(ads, msg)) { + DOM_SID group_sid; - if (!ads_pull_sid(ads, msg, "objectSid", &group_sid)) { - DEBUG(1,("No sid for this group ?!?\n")); - continue; + if (!ads_pull_sid(ads, msg, "objectSid", &group_sid)) { + DEBUG(1,("No sid for this group ?!?\n")); + continue; + } + + add_sid_to_array(mem_ctx, &group_sid, user_sids, + num_groups); } - add_sid_to_array(mem_ctx, &group_sid, user_sids, num_groups); } status = (user_sids != NULL) ? NT_STATUS_OK : NT_STATUS_NO_MEMORY; -- cgit