winbindd: remove fallback from lookuprids
authorRalph Boehme <slow@samba.org>
Sun, 2 Apr 2017 12:15:33 +0000 (14:15 +0200)
committerJeremy Allison <jra@samba.org>
Tue, 11 Apr 2017 23:41:14 +0000 (01:41 +0200)
commit8220d7453fcf900eb50a2a06b46f1c7807b02ec9
treed5ff22bfd684763da3cfe2e76916332b21723ee7
parent2eeb53890bba81a0fce18c3eb185c337c9222202
winbindd: remove fallback from lookuprids

We're only calling lookuprids for our local SAM and BUILTIN domains, if
that results in a failed lookup for some rid, sending it again via
lookupsids() won't help, it will just fail again.

If the caller wrongly had sent any other SID that is not from our SAM or
BUILTIN via lookuprids(), that it is up to the caller to fix that, not
us.

The retry logic with going through the single sids lookup at the end
added a fake domain with an empty string. The wb_lookupsids caller
wb_sids2xids needed this, as it wasn't doing the needed error handling
itself. As wb_sids2xids has been fixed to cope, we can just fail the
lookupsids here.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/winbindd/wb_lookupsids.c