r21918: Reverting this change as it is now causing aborts() in

find_builtin_domain().    This all needs more testing
before anyone starts changing these lookup routines again.

diff --git a/source/nsswitch/winbindd_util.c b/source/nsswitch/winbindd_util.c
index b649dc4768a846d775bf97b83d01b2c5e2296e9c..10fbd7c639bf7bb7ef7de7fec8512fa44355d058 100644 (file)
--- a/source/nsswitch/winbindd_util.c
+++ b/source/nsswitch/winbindd_util.c
@@ -604,13 +604,8 @@ struct winbindd_domain *find_domain_from_sid_noinit(const DOM_SID *sid)
        /* Search through list */
 
        for (domain = domain_list(); domain != NULL; domain = domain->next) {
-               /* We need to use sid_peek_check_rid, because we want 
-                * to make sure that the SIDs we send to the backends are
-                * as specific as possible.
-                */
-               if (sid_peek_check_rid(&domain->sid, sid, &discard)) {
+               if (sid_compare_domain(sid, &domain->sid) == 0)
                        return domain;
-               }
        }
 
        /* Not found */