winbindd: remove lookupsid() fallback for a failed lookupsids()
authorRalph Boehme <slow@samba.org>
Fri, 24 Mar 2017 15:54:39 +0000 (16:54 +0100)
committerJeremy Allison <jra@samba.org>
Tue, 11 Apr 2017 23:41:14 +0000 (01:41 +0200)
commit2eeb53890bba81a0fce18c3eb185c337c9222202
tree223a0d794159e98f3e82f2890f010ae238621a7b
parentc79a5acf201f1b779a4fc833a45a9ee658f8cb31
winbindd: remove lookupsid() fallback for a failed lookupsids()

If lookupsids() returned any other error then OK, SOME_NOT_MAPPED or
NONE_MAPPED we must just bail out.

If some or all SIDs could not be mapped via lookupds(), don't fallback
to lookupsid(), it will just fail again.

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