diff options
author | Jelmer Vernooij <jelmer@samba.org> | 2007-12-06 21:16:40 +0100 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2007-12-21 05:48:31 +0100 |
commit | 090d251c19e31450d8fc4af50e7c6b0c09af5f89 (patch) | |
tree | dbd0fd2af9530164f498980ebbff46e01fbcbb0f | |
parent | cc35894fb5b86373677a89f33297e763f9813ef1 (diff) | |
download | samba-090d251c19e31450d8fc4af50e7c6b0c09af5f89.tar.gz samba-090d251c19e31450d8fc4af50e7c6b0c09af5f89.tar.bz2 samba-090d251c19e31450d8fc4af50e7c6b0c09af5f89.zip |
r26318: Don't rely on SAMDB functions in secrets database.
(This used to be commit 791285f66c8f35b4f6bb9fc676bc0adc3e5b172d)
-rw-r--r-- | source4/param/secrets.c | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/source4/param/secrets.c b/source4/param/secrets.c index 5a6df3174a..0957fad5a1 100644 --- a/source4/param/secrets.c +++ b/source4/param/secrets.c @@ -30,7 +30,7 @@ #include "lib/tdb/include/tdb.h" #include "lib/util/util_tdb.h" #include "lib/util/util_ldb.h" -#include "dsdb/samdb/samdb.h" +#include "librpc/gen_ndr/ndr_security.h" static struct tdb_wrap *tdb; @@ -148,6 +148,8 @@ struct dom_sid *secrets_get_domain_sid(TALLOC_CTX *mem_ctx, int ldb_ret; const char *attrs[] = { "objectSid", NULL }; struct dom_sid *result = NULL; + const struct ldb_val *v; + enum ndr_err_code ndr_err; ldb = secrets_db_connect(mem_ctx, lp_ctx); if (ldb == NULL) { @@ -180,10 +182,22 @@ struct dom_sid *secrets_get_domain_sid(TALLOC_CTX *mem_ctx, return NULL; } - result = samdb_result_dom_sid(mem_ctx, msgs[0], "objectSid"); - if (result == NULL) { + v = ldb_msg_find_ldb_val(msgs[0], "objectSid"); + if (v == NULL) { DEBUG(0, ("Domain object for %s does not contain a SID!\n", domain)); + return NULL; + } + result = talloc(mem_ctx, struct dom_sid); + if (result == NULL) { + talloc_free(ldb); + return NULL; + } + + ndr_err = ndr_pull_struct_blob(v, result, result, + (ndr_pull_flags_fn_t)ndr_pull_dom_sid); + if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { + talloc_free(result); talloc_free(ldb); return NULL; } |