messaging4: Simplify irpc_namedb_open a bit
authorVolker Lendecke <vl@samba.org>
Thu, 27 Mar 2014 10:42:34 +0000 (10:42 +0000)
committerJeremy Allison <jra@samba.org>
Mon, 31 Mar 2014 20:52:14 +0000 (22:52 +0200)
It does not really need the msg_ctx, and it also removes the implicit talloc
hierarchy.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source4/lib/messaging/messaging.c

index fbdfd95d910fe9a5a8d6fc49bf51aa402801748e..c33aae34969348d50103869edbdbd2642120617b 100644 (file)
@@ -39,7 +39,7 @@
 /* change the message version with any incompatible changes in the protocol */
 #define IMESSAGING_VERSION 1
 
-static struct tdb_wrap *irpc_namedb_open(struct imessaging_context *msg_ctx,
+static struct tdb_wrap *irpc_namedb_open(TALLOC_CTX *mem_ctx, const char *base_path,
                                         struct loadparm_context *lp_ctx);
 
 /*
@@ -626,7 +626,7 @@ struct imessaging_context *imessaging_init(TALLOC_CTX *mem_ctx,
 
        msg->start_time    = timeval_current();
 
-       msg->names_db = irpc_namedb_open(msg, lp_ctx);
+       msg->names_db = irpc_namedb_open(msg, msg->base_path, lp_ctx);
        if (msg->names_db == NULL) {
                goto fail;
        }
@@ -907,15 +907,15 @@ static int irpc_destructor(struct irpc_request *irpc)
 /*
   open the naming database
 */
-static struct tdb_wrap *irpc_namedb_open(struct imessaging_context *msg_ctx,
+static struct tdb_wrap *irpc_namedb_open(TALLOC_CTX *mem_ctx, const char *base_path,
                                         struct loadparm_context *lp_ctx)
 {
        struct tdb_wrap *t;
-       char *path = talloc_asprintf(msg_ctx, "%s/names.tdb", msg_ctx->base_path);
+       char *path = talloc_asprintf(mem_ctx, "%s/names.tdb", base_path);
        if (path == NULL) {
                return NULL;
        }
-       t = tdb_wrap_open(msg_ctx, path, lpcfg_tdb_hash_size(lp_ctx, path),
+       t = tdb_wrap_open(mem_ctx, path, lpcfg_tdb_hash_size(lp_ctx, path),
                          lpcfg_tdb_flags(lp_ctx, 0), O_RDWR|O_CREAT, 0660);
        talloc_free(path);
        return t;