diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2004-02-25 22:09:46 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2004-02-25 22:09:46 +0000 |
| commit | ebd86253c6bfb8ccb01e23c7e0446802e9009825 (patch) | |
| tree | bfa51bf45beb253d0c4828cd05cd724438e83e5d /source3/rpc_client/cli_samr.c | |
| parent | 3f67bb3a69f7a1631e2e158f4d58095c88f6bbb2 (diff) | |
| download | samba-ebd86253c6bfb8ccb01e23c7e0446802e9009825.tar.gz samba-ebd86253c6bfb8ccb01e23c7e0446802e9009825.tar.bz2 samba-ebd86253c6bfb8ccb01e23c7e0446802e9009825.zip | |
(merge from 3.0)
I *hate* global variables...
OK, what was happening here was that we would invalidate global_sam_sid
when we set the sid into secrets.tdb, to force a re-read.
The problem was, we would do *two* writes into the TDB, and the second one
(in the PDC/BDC case) would be of a NULL pointer. This caused smbd startups
to fail, on a blank TDB.
By using a local variable in the pdb_generate_sam_sid() code, we avoid this
particular trap.
I've also added better debugging for the case where this all matters, which
is particularly for LDAP, where it finds out a domain SID from the sambaDomain
object.
Andrew Bartlett
(This used to be commit f3ecdea56d9ea6d562ace84f0e653a641eb96f6e)
Diffstat (limited to 'source3/rpc_client/cli_samr.c')
0 files changed, 0 insertions, 0 deletions