diff options
author | Andrew Bartlett <abartlet@samba.org> | 2014-02-05 15:19:41 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2014-02-05 08:42:25 +0100 |
commit | e76bbef8b796441985550c553db1ab48d6495709 (patch) | |
tree | 7df80262634c8837157d0afa74b391c69ac5cdb9 | |
parent | 262c3de3f880bb08b1220d1e755bb31365dab49b (diff) | |
download | samba-e76bbef8b796441985550c553db1ab48d6495709.tar.gz samba-e76bbef8b796441985550c553db1ab48d6495709.tar.xz samba-e76bbef8b796441985550c553db1ab48d6495709.zip |
ldb: pass module init errors back to the caller
This makes provision errors clearer in Samba, as we can now get
permission denied errors presented from LDB modules.
Andrew Bartlett
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
-rw-r--r-- | lib/ldb/common/ldb.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/ldb/common/ldb.c b/lib/ldb/common/ldb.c index e5fa81977a7..36f1c37437c 100644 --- a/lib/ldb/common/ldb.c +++ b/lib/ldb/common/ldb.c @@ -257,11 +257,12 @@ int ldb_connect(struct ldb_context *ldb, const char *url, return ret; } - if (ldb_load_modules(ldb, options) != LDB_SUCCESS) { + ret = ldb_load_modules(ldb, options); + if (ret != LDB_SUCCESS) { ldb_debug(ldb, LDB_DEBUG_FATAL, "Unable to load modules for %s: %s", url, ldb_errstring(ldb)); - return LDB_ERR_OTHER; + return ret; } /* set the default base dn */ |