s3/winbindd: Remove unnecessary check for NULL SID
commit5469866242f0dfbdc7260c86cfca196b10e4af21
authorSteven Danneman <steven.danneman@isilon.com>
Wed, 29 Jul 2009 23:13:44 +0000 (29 16:13 -0700)
committerSteven Danneman <steven.danneman@isilon.com>
Wed, 26 Aug 2009 23:34:09 +0000 (26 16:34 -0700)
tree664c544d72ae38b273eeb340f7e9d00e3cad6f42
parent32f9d20dff043cca5b81ff855c44a71bf18e41e4
s3/winbindd: Remove unnecessary check for NULL SID

There's a known bug in some Windows implementations of
DsEnumerateDomainTrusts() where domain SIDs are not returned for
transitively trusted domains within the same forest.

Jerry originally worked around this in the winbindd parent by checking
for S-0-0 and converting it to S-1-0 in 8b0fce0b.  Guenter later moved
these checks into the child process in commit 3bdfcbac making the
initial patch unecessary.

I've removed it and added a clarifying comment to the child process.

If ever this SID is needed we could add an extra DsEnumerateDomainTrusts()
call in trusted_domains() as suggested by the Microsoft KB.
source3/winbindd/winbindd_ads.c
source3/winbindd/winbindd_util.c