LookupAccountNameW() use ACCOUNT_SIDS domain and name_use fields for well known SIDs
Paul Bryan Roberts
pbronline-wine at yahoo.co.uk
Fri Sep 26 05:52:43 CDT 2008
LookupAccountNameW() has a stubbed but non-empty, incomplete, buggy
implementation. It fails a number of unit tests but Wine code relies on
this.
This is the FOURTH of a series of NINE patches that provide what may be
a reasonable first pass implementation of LookupAccountNameA() and
LookupAccountNameW() that:
- works for well known SIDs
- works for 'first user account'
- adds handling of 'first user account' to LookupAccountSid
- passes existing units tests for LookupAccountName
- corrects knock-on regression failures in the advapit32/security.c
unit tests (there may be others elsewhere)
- allows progress towards the resolution of Bug 14334
This patch replace the stub implementation with code that uses the
ACCOUNT_SIDS domain and name_use fields.
This patch does not fix any units tests, nor does it break any.
Perhaps I should have added some but the only tests I could add would be
to try all well-known-SIDS, which would have introduced a load more
failing units tests and would have simply have been a distraction from
the business at hand.
While this patch may not be obviously incorrect, it is not correct and
is amended later when we have a unit test failure to correct.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0007-LookupAccountNameW-use-ACCOUNT_SIDS-domain-and-name_.txt
Url: http://www.winehq.org/pipermail/wine-patches/attachments/20080926/828318db/attachment.txt
More information about the wine-patches
mailing list