ctdb-daemon: Rename struct ctdb_client_notify_register to ctdb_notify_data_old
[obnox/samba/samba-obnox.git] / source3 / lib / ctdbd_conn.c
index 1bb9125f69ffbc8e91f5e78defecf9a7a5a51440..e67a31d5bc6becce12a345abf3af56b89614b970 100644 (file)
 #include "serverid.h"
 #include "ctdbd_conn.h"
 #include "system/select.h"
-#include "lib/sys_rw_data.h"
+#include "lib/util/sys_rw_data.h"
 #include "lib/util/iov_buf.h"
 
 #include "messages.h"
 
 /* paths to these include files come from --with-ctdb= in configure */
 
-#include "ctdb.h"
 #include "ctdb_private.h"
 
 struct ctdbd_srvid_cb {
@@ -63,11 +62,12 @@ static uint32_t ctdbd_next_reqid(struct ctdbd_connection *conn)
        return conn->reqid;
 }
 
-static NTSTATUS ctdbd_control(struct ctdbd_connection *conn,
-                             uint32_t vnn, uint32_t opcode,
-                             uint64_t srvid, uint32_t flags, TDB_DATA data,
-                             TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
-                             int *cstatus);
+static int ctdbd_control(struct ctdbd_connection *conn,
+                        uint32_t vnn, uint32_t opcode,
+                        uint64_t srvid, uint32_t flags,
+                        TDB_DATA data,
+                        TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
+                        int *cstatus);
 
 /*
  * exit on fatal communications errors with the ctdbd daemon
@@ -99,24 +99,23 @@ static void ctdb_packet_dump(struct ctdb_req_header *hdr)
 /*
  * Register a srvid with ctdbd
  */
-NTSTATUS register_with_ctdbd(struct ctdbd_connection *conn, uint64_t srvid,
-                            int (*cb)(uint32_t src_vnn, uint32_t dst_vnn,
-                                      uint64_t dst_srvid,
-                                      const uint8_t *msg, size_t msglen,
-                                      void *private_data),
-                            void *private_data)
+int register_with_ctdbd(struct ctdbd_connection *conn, uint64_t srvid,
+                       int (*cb)(uint32_t src_vnn, uint32_t dst_vnn,
+                                 uint64_t dst_srvid,
+                                 const uint8_t *msg, size_t msglen,
+                                 void *private_data),
+                       void *private_data)
 {
 
-       NTSTATUS status;
-       int cstatus;
+       int ret, cstatus;
        size_t num_callbacks;
        struct ctdbd_srvid_cb *tmp;
 
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              CTDB_CONTROL_REGISTER_SRVID, srvid, 0,
-                              tdb_null, NULL, NULL, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_REGISTER_SRVID, srvid, 0,
+                           tdb_null, NULL, NULL, &cstatus);
+       if (ret != 0) {
+               return ret;
        }
 
        num_callbacks = talloc_array_length(conn->callbacks);
@@ -124,7 +123,7 @@ NTSTATUS register_with_ctdbd(struct ctdbd_connection *conn, uint64_t srvid,
        tmp = talloc_realloc(conn, conn->callbacks, struct ctdbd_srvid_cb,
                             num_callbacks + 1);
        if (tmp == NULL) {
-               return NT_STATUS_NO_MEMORY;
+               return ENOMEM;
        }
        conn->callbacks = tmp;
 
@@ -132,22 +131,22 @@ NTSTATUS register_with_ctdbd(struct ctdbd_connection *conn, uint64_t srvid,
                .srvid = srvid, .cb = cb, .private_data = private_data
        };
 
-       return NT_STATUS_OK;
+       return 0;
 }
 
 static int ctdbd_msg_call_back(struct ctdbd_connection *conn,
-                              struct ctdb_req_message *msg)
+                              struct ctdb_req_message_old *msg)
 {
        size_t msg_len;
        size_t i, num_callbacks;
 
        msg_len = msg->hdr.length;
-       if (msg_len < offsetof(struct ctdb_req_message, data)) {
+       if (msg_len < offsetof(struct ctdb_req_message_old, data)) {
                DEBUG(10, ("%s: len %u too small\n", __func__,
                           (unsigned)msg_len));
                return 0;
        }
-       msg_len -= offsetof(struct ctdb_req_message, data);
+       msg_len -= offsetof(struct ctdb_req_message_old, data);
 
        if (msg_len < msg->datalen) {
                DEBUG(10, ("%s: msg_len=%u < msg->datalen=%u\n", __func__,
@@ -177,19 +176,19 @@ static int ctdbd_msg_call_back(struct ctdbd_connection *conn,
 /*
  * get our vnn from the cluster
  */
-static NTSTATUS get_cluster_vnn(struct ctdbd_connection *conn, uint32_t *vnn)
+static int get_cluster_vnn(struct ctdbd_connection *conn, uint32_t *vnn)
 {
        int32_t cstatus=-1;
-       NTSTATUS status;
-       status = ctdbd_control(conn,
-                              CTDB_CURRENT_NODE, CTDB_CONTROL_GET_PNN, 0, 0,
-                              tdb_null, NULL, NULL, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(1, ("ctdbd_control failed: %s\n", nt_errstr(status)));
-               return status;
+       int ret;
+       ret = ctdbd_control(conn,
+                           CTDB_CURRENT_NODE, CTDB_CONTROL_GET_PNN, 0, 0,
+                           tdb_null, NULL, NULL, &cstatus);
+       if (ret != 0) {
+               DEBUG(1, ("ctdbd_control failed: %s\n", strerror(ret)));
+               return ret;
        }
        *vnn = (uint32_t)cstatus;
-       return status;
+       return ret;
 }
 
 /*
@@ -198,18 +197,17 @@ static NTSTATUS get_cluster_vnn(struct ctdbd_connection *conn, uint32_t *vnn)
 static bool ctdbd_working(struct ctdbd_connection *conn, uint32_t vnn)
 {
        int32_t cstatus=-1;
-       NTSTATUS status;
        TDB_DATA outdata;
-       struct ctdb_node_map *m;
+       struct ctdb_node_map_old *m;
        uint32_t failure_flags;
-       bool ret = false;
-       int i;
-
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              CTDB_CONTROL_GET_NODEMAP, 0, 0,
-                              tdb_null, talloc_tos(), &outdata, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(1, ("ctdbd_control failed: %s\n", nt_errstr(status)));
+       bool ok = false;
+       int i, ret;
+
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_GET_NODEMAP, 0, 0,
+                           tdb_null, talloc_tos(), &outdata, &cstatus);
+       if (ret != 0) {
+               DEBUG(1, ("ctdbd_control failed: %s\n", strerror(ret)));
                return false;
        }
        if ((cstatus != 0) || (outdata.dptr == NULL)) {
@@ -217,7 +215,7 @@ static bool ctdbd_working(struct ctdbd_connection *conn, uint32_t vnn)
                return false;
        }
 
-       m = (struct ctdb_node_map *)outdata.dptr;
+       m = (struct ctdb_node_map_old *)outdata.dptr;
 
        for (i=0; i<m->num; i++) {
                if (vnn == m->nodes[i].pnn) {
@@ -240,10 +238,10 @@ static bool ctdbd_working(struct ctdbd_connection *conn, uint32_t vnn)
                goto fail;
        }
 
-       ret = true;
+       ok = true;
 fail:
        TALLOC_FREE(outdata.dptr);
-       return ret;
+       return ok;
 }
 
 uint32_t ctdbd_vnn(const struct ctdbd_connection *conn)
@@ -251,18 +249,6 @@ uint32_t ctdbd_vnn(const struct ctdbd_connection *conn)
        return conn->our_vnn;
 }
 
-const char *lp_ctdbd_socket(void)
-{
-       const char *ret;
-
-       ret = lp__ctdbd_socket();
-       if (ret != NULL && strlen(ret) > 0) {
-               return ret;
-       }
-
-       return CTDB_SOCKET;
-}
-
 /*
  * Get us a ctdb connection
  */
@@ -314,7 +300,7 @@ static int ctdb_read_packet(int fd, int timeout, TALLOC_CTX *mem_ctx,
        ssize_t nread;
 
        if (timeout != -1) {
-               ret = poll_one_fd(fd, POLLIN, timeout, &revents);
+               ret = poll_intr_one_fd(fd, POLLIN, timeout, &revents);
                if (ret == -1) {
                        return errno;
                }
@@ -349,9 +335,11 @@ static int ctdb_read_packet(int fd, int timeout, TALLOC_CTX *mem_ctx,
        nread = read_data(fd, ((char *)req) + sizeof(msglen),
                          msglen - sizeof(msglen));
        if (nread == -1) {
+               TALLOC_FREE(req);
                return errno;
        }
        if (nread == 0) {
+               TALLOC_FREE(req);
                return EIO;
        }
 
@@ -382,7 +370,7 @@ static int ctdb_read_req(struct ctdbd_connection *conn, uint32_t reqid,
        ctdb_packet_dump(hdr);
 
        if (hdr->operation == CTDB_REQ_MESSAGE) {
-               struct ctdb_req_message *msg = (struct ctdb_req_message *)hdr;
+               struct ctdb_req_message_old *msg = (struct ctdb_req_message_old *)hdr;
 
                if (conn->msg_ctx == NULL) {
                        DEBUG(1, ("Got a message without having a msg ctx, "
@@ -417,30 +405,33 @@ static int ctdb_read_req(struct ctdbd_connection *conn, uint32_t reqid,
 
 static int ctdbd_connection_destructor(struct ctdbd_connection *c)
 {
-       close(c->fd);
+       TALLOC_FREE(c->fde);
+       if (c->fd != -1) {
+               close(c->fd);
+               c->fd = -1;
+       }
        return 0;
 }
 /*
  * Get us a ctdbd connection
  */
 
-static NTSTATUS ctdbd_init_connection(TALLOC_CTX *mem_ctx,
-                                     const char *sockname, int timeout,
-                                     struct ctdbd_connection **pconn)
+static int ctdbd_init_connection(TALLOC_CTX *mem_ctx,
+                                const char *sockname, int timeout,
+                                struct ctdbd_connection **pconn)
 {
        struct ctdbd_connection *conn;
        int ret;
-       NTSTATUS status;
 
        if (!(conn = talloc_zero(mem_ctx, struct ctdbd_connection))) {
                DEBUG(0, ("talloc failed\n"));
-               return NT_STATUS_NO_MEMORY;
+               return ENOMEM;
        }
 
        conn->sockname = talloc_strdup(conn, sockname);
        if (conn->sockname == NULL) {
-               DBG_ERR("%s: talloc failed\n", __func__);
-               status = NT_STATUS_NO_MEMORY;
+               DBG_ERR("talloc failed\n");
+               ret = ENOMEM;
                goto fail;
        }
 
@@ -452,72 +443,71 @@ static NTSTATUS ctdbd_init_connection(TALLOC_CTX *mem_ctx,
 
        ret = ctdbd_connect(conn->sockname, &conn->fd);
        if (ret != 0) {
-               status = map_nt_error_from_unix(ret);
                DEBUG(1, ("ctdbd_connect failed: %s\n", strerror(ret)));
                goto fail;
        }
        talloc_set_destructor(conn, ctdbd_connection_destructor);
 
-       status = get_cluster_vnn(conn, &conn->our_vnn);
+       ret = get_cluster_vnn(conn, &conn->our_vnn);
 
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("get_cluster_vnn failed: %s\n", nt_errstr(status)));
+       if (ret != 0) {
+               DEBUG(10, ("get_cluster_vnn failed: %s\n", strerror(ret)));
                goto fail;
        }
 
        if (!ctdbd_working(conn, conn->our_vnn)) {
                DEBUG(2, ("Node is not working, can not connect\n"));
-               status = NT_STATUS_INTERNAL_DB_ERROR;
+               ret = EIO;
                goto fail;
        }
 
        generate_random_buffer((unsigned char *)&conn->rand_srvid,
                               sizeof(conn->rand_srvid));
 
-       status = register_with_ctdbd(conn, conn->rand_srvid, NULL, NULL);
+       ret = register_with_ctdbd(conn, conn->rand_srvid, NULL, NULL);
 
-       if (!NT_STATUS_IS_OK(status)) {
+       if (ret != 0) {
                DEBUG(5, ("Could not register random srvid: %s\n",
-                         nt_errstr(status)));
+                         strerror(ret)));
                goto fail;
        }
 
        *pconn = conn;
-       return NT_STATUS_OK;
+       return 0;
 
  fail:
        TALLOC_FREE(conn);
-       return status;
+       return ret;
 }
 
 /*
  * Get us a ctdbd connection and register us as a process
  */
 
-NTSTATUS ctdbd_messaging_connection(TALLOC_CTX *mem_ctx,
-                                   struct ctdbd_connection **pconn)
+int ctdbd_messaging_connection(TALLOC_CTX *mem_ctx,
+                              const char *sockname, int timeout,
+                              struct ctdbd_connection **pconn)
 {
         struct ctdbd_connection *conn;
-       NTSTATUS status;
+       int ret;
 
-       status = ctdbd_init_connection(mem_ctx, lp_ctdbd_socket(),
-                                      lp_ctdb_timeout(), &conn);
+       ret = ctdbd_init_connection(mem_ctx, sockname, timeout, &conn);
 
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
+       if (ret != 0) {
+               return ret;
        }
 
-       status = register_with_ctdbd(conn, MSG_SRVID_SAMBA, NULL, NULL);
-       if (!NT_STATUS_IS_OK(status)) {
+       ret = register_with_ctdbd(conn, MSG_SRVID_SAMBA, NULL, NULL);
+       if (ret != 0) {
                goto fail;
        }
 
        *pconn = conn;
-       return NT_STATUS_OK;
+       return 0;
 
  fail:
        TALLOC_FREE(conn);
-       return status;
+       return ret;
 }
 
 struct messaging_context *ctdb_conn_msg_ctx(struct ctdbd_connection *conn)
@@ -536,7 +526,7 @@ int ctdbd_conn_get_fd(struct ctdbd_connection *conn)
 static int ctdb_handle_message(struct ctdbd_connection *conn,
                               struct ctdb_req_header *hdr)
 {
-       struct ctdb_req_message *msg;
+       struct ctdb_req_message_old *msg;
 
        if (hdr->operation != CTDB_REQ_MESSAGE) {
                DEBUG(0, ("Received async msg of type %u, discarding\n",
@@ -544,7 +534,7 @@ static int ctdb_handle_message(struct ctdbd_connection *conn,
                return EINVAL;
        }
 
-       msg = (struct ctdb_req_message *)hdr;
+       msg = (struct ctdb_req_message_old *)hdr;
 
        ctdbd_msg_call_back(conn, msg);
 
@@ -585,37 +575,35 @@ static void ctdbd_socket_handler(struct tevent_context *event_ctx,
  * Prepare a ctdbd connection to receive messages
  */
 
-NTSTATUS ctdbd_register_msg_ctx(struct ctdbd_connection *conn,
-                               struct messaging_context *msg_ctx)
+int ctdbd_register_msg_ctx(struct ctdbd_connection *conn,
+                          struct messaging_context *msg_ctx,
+                          struct tevent_context *ev)
 {
        SMB_ASSERT(conn->msg_ctx == NULL);
        SMB_ASSERT(conn->fde == NULL);
 
-       if (!(conn->fde = tevent_add_fd(messaging_tevent_context(msg_ctx),
-                                      conn,
-                                      conn->fd,
-                                      TEVENT_FD_READ,
-                                      ctdbd_socket_handler,
-                                      conn))) {
+       conn->fde = tevent_add_fd(ev, conn, conn->fd, TEVENT_FD_READ,
+                                 ctdbd_socket_handler, conn);
+       if (conn->fde == NULL) {
                DEBUG(0, ("event_add_fd failed\n"));
-               return NT_STATUS_NO_MEMORY;
+               return ENOMEM;
        }
 
        conn->msg_ctx = msg_ctx;
 
-       return NT_STATUS_OK;
+       return 0;
 }
 
-NTSTATUS ctdbd_messaging_send_iov(struct ctdbd_connection *conn,
-                                 uint32_t dst_vnn, uint64_t dst_srvid,
-                                 const struct iovec *iov, int iovlen)
+int ctdbd_messaging_send_iov(struct ctdbd_connection *conn,
+                            uint32_t dst_vnn, uint64_t dst_srvid,
+                            const struct iovec *iov, int iovlen)
 {
-       struct ctdb_req_message r;
+       struct ctdb_req_message_old r;
        struct iovec iov2[iovlen+1];
        size_t buflen = iov_buflen(iov, iovlen);
        ssize_t nwritten;
 
-       r.hdr.length = offsetof(struct ctdb_req_message, data) + buflen;
+       r.hdr.length = offsetof(struct ctdb_req_message_old, data) + buflen;
        r.hdr.ctdb_magic = CTDB_MAGIC;
        r.hdr.ctdb_version = CTDB_PROTOCOL;
        r.hdr.generation = 1;
@@ -630,7 +618,7 @@ NTSTATUS ctdbd_messaging_send_iov(struct ctdbd_connection *conn,
        ctdb_packet_dump(&r.hdr);
 
        iov2[0].iov_base = &r;
-       iov2[0].iov_len = offsetof(struct ctdb_req_message, data);
+       iov2[0].iov_len = offsetof(struct ctdb_req_message_old, data);
        memcpy(&iov2[1], iov, iovlen * sizeof(struct iovec));
 
        nwritten = write_data_iov(conn->fd, iov2, iovlen+1);
@@ -639,29 +627,28 @@ NTSTATUS ctdbd_messaging_send_iov(struct ctdbd_connection *conn,
                cluster_fatal("cluster dispatch daemon msg write error\n");
        }
 
-       return NT_STATUS_OK;
+       return 0;
 }
 
 /*
  * send/recv a generic ctdb control message
  */
-static NTSTATUS ctdbd_control(struct ctdbd_connection *conn,
-                             uint32_t vnn, uint32_t opcode,
-                             uint64_t srvid, uint32_t flags,
-                             TDB_DATA data,
-                             TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
-                             int *cstatus)
+static int ctdbd_control(struct ctdbd_connection *conn,
+                        uint32_t vnn, uint32_t opcode,
+                        uint64_t srvid, uint32_t flags,
+                        TDB_DATA data,
+                        TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
+                        int *cstatus)
 {
-       struct ctdb_req_control req;
+       struct ctdb_req_control_old req;
        struct ctdb_req_header *hdr;
-       struct ctdb_reply_control *reply = NULL;
+       struct ctdb_reply_control_old *reply = NULL;
        struct iovec iov[2];
        ssize_t nwritten;
-       NTSTATUS status;
        int ret;
 
        ZERO_STRUCT(req);
-       req.hdr.length = offsetof(struct ctdb_req_control, data) + data.dsize;
+       req.hdr.length = offsetof(struct ctdb_req_control_old, data) + data.dsize;
        req.hdr.ctdb_magic   = CTDB_MAGIC;
        req.hdr.ctdb_version = CTDB_PROTOCOL;
        req.hdr.operation    = CTDB_REQ_CONTROL;
@@ -676,7 +663,7 @@ static NTSTATUS ctdbd_control(struct ctdbd_connection *conn,
        ctdb_packet_dump(&req.hdr);
 
        iov[0].iov_base = &req;
-       iov[0].iov_len = offsetof(struct ctdb_req_control, data);
+       iov[0].iov_len = offsetof(struct ctdb_req_control_old, data);
        iov[1].iov_base = data.dptr;
        iov[1].iov_len = data.dsize;
 
@@ -690,27 +677,27 @@ static NTSTATUS ctdbd_control(struct ctdbd_connection *conn,
                if (cstatus) {
                        *cstatus = 0;
                }
-               return NT_STATUS_OK;
+               return 0;
        }
 
        ret = ctdb_read_req(conn, req.hdr.reqid, NULL, &hdr);
        if (ret != 0) {
                DEBUG(10, ("ctdb_read_req failed: %s\n", strerror(ret)));
-               status = map_nt_error_from_unix(ret);
-               goto fail;
+               return ret;
        }
 
        if (hdr->operation != CTDB_REPLY_CONTROL) {
                DEBUG(0, ("received invalid reply\n"));
-               goto fail;
+               TALLOC_FREE(hdr);
+               return EIO;
        }
-       reply = (struct ctdb_reply_control *)hdr;
+       reply = (struct ctdb_reply_control_old *)hdr;
 
        if (outdata) {
                if (!(outdata->dptr = (uint8_t *)talloc_memdup(
                              mem_ctx, reply->data, reply->datalen))) {
                        TALLOC_FREE(reply);
-                       return NT_STATUS_NO_MEMORY;
+                       return ENOMEM;
                }
                outdata->dsize = reply->datalen;
        }
@@ -718,11 +705,8 @@ static NTSTATUS ctdbd_control(struct ctdbd_connection *conn,
                (*cstatus) = reply->status;
        }
 
-       status = NT_STATUS_OK;
-
- fail:
        TALLOC_FREE(reply);
-       return status;
+       return ret;
 }
 
 /*
@@ -758,7 +742,7 @@ bool ctdb_processes_exist(struct ctdbd_connection *conn,
        }
 
        for (i=0; i<num_pids; i++) {
-               struct ctdb_req_control req;
+               struct ctdb_req_control_old req;
                pid_t pid;
                struct iovec iov[2];
                ssize_t nwritten;
@@ -778,7 +762,7 @@ bool ctdb_processes_exist(struct ctdbd_connection *conn,
                           (int)pids[i].vnn, (int)pid,
                           (int)reqids[i]));
 
-               req.hdr.length = offsetof(struct ctdb_req_control, data);
+               req.hdr.length = offsetof(struct ctdb_req_control_old, data);
                req.hdr.length += sizeof(pid);
                req.hdr.ctdb_magic   = CTDB_MAGIC;
                req.hdr.ctdb_version = CTDB_PROTOCOL;
@@ -794,7 +778,7 @@ bool ctdb_processes_exist(struct ctdbd_connection *conn,
                ctdb_packet_dump(&req.hdr);
 
                iov[0].iov_base = &req;
-               iov[0].iov_len = offsetof(struct ctdb_req_control, data);
+               iov[0].iov_len = offsetof(struct ctdb_req_control_old, data);
                iov[1].iov_base = &pid;
                iov[1].iov_len = sizeof(pid);
 
@@ -810,7 +794,7 @@ bool ctdb_processes_exist(struct ctdbd_connection *conn,
 
        while (num_received < num_pids) {
                struct ctdb_req_header *hdr;
-               struct ctdb_reply_control *reply;
+               struct ctdb_reply_control_old *reply;
                uint32_t reqid;
                int ret;
 
@@ -825,7 +809,7 @@ bool ctdb_processes_exist(struct ctdbd_connection *conn,
                        DEBUG(10, ("Received invalid reply\n"));
                        goto fail;
                }
-               reply = (struct ctdb_reply_control *)hdr;
+               reply = (struct ctdb_reply_control_old *)hdr;
 
                reqid = reply->hdr.reqid;
 
@@ -858,7 +842,7 @@ fail:
 char *ctdbd_dbpath(struct ctdbd_connection *conn,
                   TALLOC_CTX *mem_ctx, uint32_t db_id)
 {
-       NTSTATUS status;
+       int ret;
        TDB_DATA data;
        TDB_DATA rdata = {0};
        int32_t cstatus = 0;
@@ -866,11 +850,12 @@ char *ctdbd_dbpath(struct ctdbd_connection *conn,
        data.dptr = (uint8_t*)&db_id;
        data.dsize = sizeof(db_id);
 
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              CTDB_CONTROL_GETDBPATH, 0, 0, data,
-                              mem_ctx, &rdata, &cstatus);
-       if (!NT_STATUS_IS_OK(status) || cstatus != 0) {
-               DEBUG(0,(__location__ " ctdb_control for getdbpath failed\n"));
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_GETDBPATH, 0, 0, data,
+                           mem_ctx, &rdata, &cstatus);
+       if ((ret != 0) || cstatus != 0) {
+               DEBUG(0, (__location__ " ctdb_control for getdbpath failed: %s\n",
+                         strerror(ret)));
                return NULL;
        }
 
@@ -880,71 +865,68 @@ char *ctdbd_dbpath(struct ctdbd_connection *conn,
 /*
  * attach to a ctdb database
  */
-NTSTATUS ctdbd_db_attach(struct ctdbd_connection *conn,
-                        const char *name, uint32_t *db_id, int tdb_flags)
+int ctdbd_db_attach(struct ctdbd_connection *conn,
+                   const char *name, uint32_t *db_id, int tdb_flags)
 {
-       NTSTATUS status;
+       int ret;
        TDB_DATA data;
        int32_t cstatus;
        bool persistent = (tdb_flags & TDB_CLEAR_IF_FIRST) == 0;
 
        data = string_term_tdb_data(name);
 
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              persistent
-                              ? CTDB_CONTROL_DB_ATTACH_PERSISTENT
-                              : CTDB_CONTROL_DB_ATTACH,
-                              tdb_flags, 0, data, NULL, &data, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           persistent
+                           ? CTDB_CONTROL_DB_ATTACH_PERSISTENT
+                           : CTDB_CONTROL_DB_ATTACH,
+                           tdb_flags, 0, data, NULL, &data, &cstatus);
+       if (ret != 0) {
                DEBUG(0, (__location__ " ctdb_control for db_attach "
-                         "failed: %s\n", nt_errstr(status)));
-               return status;
+                         "failed: %s\n", strerror(ret)));
+               return ret;
        }
 
        if (cstatus != 0 || data.dsize != sizeof(uint32_t)) {
                DEBUG(0,(__location__ " ctdb_control for db_attach failed\n"));
-               return NT_STATUS_INTERNAL_ERROR;
+               return EIO;
        }
 
        *db_id = *(uint32_t *)data.dptr;
        talloc_free(data.dptr);
 
        if (!(tdb_flags & TDB_SEQNUM)) {
-               return NT_STATUS_OK;
+               return 0;
        }
 
        data.dptr = (uint8_t *)db_id;
        data.dsize = sizeof(*db_id);
 
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              CTDB_CONTROL_ENABLE_SEQNUM, 0, 0, data,
-                              NULL, NULL, &cstatus);
-       if (!NT_STATUS_IS_OK(status) || cstatus != 0) {
-               DEBUG(0,(__location__ " ctdb_control for enable seqnum "
-                        "failed\n"));
-               return NT_STATUS_IS_OK(status) ? NT_STATUS_INTERNAL_ERROR :
-                       status;
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_ENABLE_SEQNUM, 0, 0, data,
+                           NULL, NULL, &cstatus);
+       if ((ret != 0) || cstatus != 0) {
+               DEBUG(0, (__location__ " ctdb_control for enable seqnum "
+                         "failed: %s\n", strerror(ret)));
+               return (ret == 0) ? EIO : ret;
        }
 
-       return NT_STATUS_OK;
+       return 0;
 }
 
 /*
  * force the migration of a record to this node
  */
-NTSTATUS ctdbd_migrate(struct ctdbd_connection *conn, uint32_t db_id,
-                      TDB_DATA key)
+int ctdbd_migrate(struct ctdbd_connection *conn, uint32_t db_id, TDB_DATA key)
 {
-       struct ctdb_req_call req;
+       struct ctdb_req_call_old req;
        struct ctdb_req_header *hdr;
        struct iovec iov[2];
        ssize_t nwritten;
-       NTSTATUS status;
        int ret;
 
        ZERO_STRUCT(req);
 
-       req.hdr.length = offsetof(struct ctdb_req_call, data) + key.dsize;
+       req.hdr.length = offsetof(struct ctdb_req_call_old, data) + key.dsize;
        req.hdr.ctdb_magic   = CTDB_MAGIC;
        req.hdr.ctdb_version = CTDB_PROTOCOL;
        req.hdr.operation    = CTDB_REQ_CALL;
@@ -958,7 +940,7 @@ NTSTATUS ctdbd_migrate(struct ctdbd_connection *conn, uint32_t db_id,
        ctdb_packet_dump(&req.hdr);
 
        iov[0].iov_base = &req;
-       iov[0].iov_len = offsetof(struct ctdb_req_call, data);
+       iov[0].iov_len = offsetof(struct ctdb_req_call_old, data);
        iov[1].iov_base = key.dptr;
        iov[1].iov_len = key.dsize;
 
@@ -971,38 +953,34 @@ NTSTATUS ctdbd_migrate(struct ctdbd_connection *conn, uint32_t db_id,
        ret = ctdb_read_req(conn, req.hdr.reqid, NULL, &hdr);
        if (ret != 0) {
                DEBUG(10, ("ctdb_read_req failed: %s\n", strerror(ret)));
-               status = map_nt_error_from_unix(ret);
                goto fail;
        }
 
        if (hdr->operation != CTDB_REPLY_CALL) {
                DEBUG(0, ("received invalid reply\n"));
-               status = NT_STATUS_INTERNAL_ERROR;
                goto fail;
        }
 
-       status = NT_STATUS_OK;
  fail:
 
        TALLOC_FREE(hdr);
-       return status;
+       return ret;
 }
 
 /*
  * Fetch a record and parse it
  */
-NTSTATUS ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
-                    TDB_DATA key, bool local_copy,
-                    void (*parser)(TDB_DATA key, TDB_DATA data,
-                                   void *private_data),
-                    void *private_data)
+int ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
+               TDB_DATA key, bool local_copy,
+               void (*parser)(TDB_DATA key, TDB_DATA data,
+                              void *private_data),
+               void *private_data)
 {
-       struct ctdb_req_call req;
+       struct ctdb_req_call_old req;
        struct ctdb_req_header *hdr = NULL;
-       struct ctdb_reply_call *reply;
+       struct ctdb_reply_call_old *reply;
        struct iovec iov[2];
        ssize_t nwritten;
-       NTSTATUS status;
        uint32_t flags;
        int ret;
 
@@ -1010,7 +988,7 @@ NTSTATUS ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
 
        ZERO_STRUCT(req);
 
-       req.hdr.length = offsetof(struct ctdb_req_call, data) + key.dsize;
+       req.hdr.length = offsetof(struct ctdb_req_call_old, data) + key.dsize;
        req.hdr.ctdb_magic   = CTDB_MAGIC;
        req.hdr.ctdb_version = CTDB_PROTOCOL;
        req.hdr.operation    = CTDB_REQ_CALL;
@@ -1021,7 +999,7 @@ NTSTATUS ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
        req.keylen           = key.dsize;
 
        iov[0].iov_base = &req;
-       iov[0].iov_len = offsetof(struct ctdb_req_call, data);
+       iov[0].iov_len = offsetof(struct ctdb_req_call_old, data);
        iov[1].iov_base = key.dptr;
        iov[1].iov_len = key.dsize;
 
@@ -1034,32 +1012,31 @@ NTSTATUS ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
        ret = ctdb_read_req(conn, req.hdr.reqid, NULL, &hdr);
        if (ret != 0) {
                DEBUG(10, ("ctdb_read_req failed: %s\n", strerror(ret)));
-               status = map_nt_error_from_unix(ret);
                goto fail;
        }
 
        if ((hdr == NULL) || (hdr->operation != CTDB_REPLY_CALL)) {
                DEBUG(0, ("received invalid reply\n"));
-               status = NT_STATUS_INTERNAL_ERROR;
+               ret = EIO;
                goto fail;
        }
-       reply = (struct ctdb_reply_call *)hdr;
+       reply = (struct ctdb_reply_call_old *)hdr;
 
        if (reply->datalen == 0) {
                /*
                 * Treat an empty record as non-existing
                 */
-               status = NT_STATUS_NOT_FOUND;
+               ret = ENOENT;
                goto fail;
        }
 
        parser(key, make_tdb_data(&reply->data[0], reply->datalen),
               private_data);
 
-       status = NT_STATUS_OK;
+       ret = 0;
  fail:
        TALLOC_FREE(hdr);
-       return status;
+       return ret;
 }
 
 /*
@@ -1068,26 +1045,25 @@ NTSTATUS ctdbd_parse(struct ctdbd_connection *conn, uint32_t db_id,
   everything in-line.
 */
 
-NTSTATUS ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
+int ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
                        void (*fn)(TDB_DATA key, TDB_DATA data,
                                   void *private_data),
                        void *private_data)
 {
        struct ctdbd_connection *conn;
-       NTSTATUS status;
-
+       int ret;
        TDB_DATA key, data;
        struct ctdb_traverse_start t;
        int cstatus;
 
        become_root();
-       status = ctdbd_init_connection(NULL, master->sockname, master->timeout,
-                                      &conn);
+       ret = ctdbd_init_connection(NULL, master->sockname, master->timeout,
+                                   &conn);
        unbecome_root();
-       if (!NT_STATUS_IS_OK(status)) {
+       if (ret != 0) {
                DEBUG(0, ("ctdbd_init_connection failed: %s\n",
-                         nt_errstr(status)));
-               return status;
+                         strerror(ret)));
+               return ret;
        }
 
        t.db_id = db_id;
@@ -1097,30 +1073,28 @@ NTSTATUS ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
        data.dptr = (uint8_t *)&t;
        data.dsize = sizeof(t);
 
-       status = ctdbd_control(conn, CTDB_CURRENT_NODE,
-                              CTDB_CONTROL_TRAVERSE_START, conn->rand_srvid, 0,
-                              data, NULL, NULL, &cstatus);
-
-       if (!NT_STATUS_IS_OK(status) || (cstatus != 0)) {
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_TRAVERSE_START, conn->rand_srvid,
+                           0, data, NULL, NULL, &cstatus);
 
-               DEBUG(0,("ctdbd_control failed: %s, %d\n", nt_errstr(status),
+       if ((ret != 0) || (cstatus != 0)) {
+               DEBUG(0,("ctdbd_control failed: %s, %d\n", strerror(ret),
                         cstatus));
 
-               if (NT_STATUS_IS_OK(status)) {
+               if (ret == 0) {
                        /*
                         * We need a mapping here
                         */
-                       status = NT_STATUS_UNSUCCESSFUL;
+                       ret = EIO;
                }
                TALLOC_FREE(conn);
-               return status;
+               return ret;
        }
 
        while (True) {
                struct ctdb_req_header *hdr = NULL;
-               struct ctdb_req_message *m;
-               struct ctdb_rec_data *d;
-               int ret;
+               struct ctdb_req_message_old *m;
+               struct ctdb_rec_data_old *d;
 
                ret = ctdb_read_packet(conn->fd, conn->timeout, conn, &hdr);
                if (ret != 0) {
@@ -1133,16 +1107,16 @@ NTSTATUS ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
                        DEBUG(0, ("Got operation %u, expected a message\n",
                                  (unsigned)hdr->operation));
                        TALLOC_FREE(conn);
-                       return NT_STATUS_UNEXPECTED_IO_ERROR;
+                       return EIO;
                }
 
-               m = (struct ctdb_req_message *)hdr;
-               d = (struct ctdb_rec_data *)&m->data[0];
+               m = (struct ctdb_req_message_old *)hdr;
+               d = (struct ctdb_rec_data_old *)&m->data[0];
                if (m->datalen < sizeof(uint32_t) || m->datalen != d->length) {
                        DEBUG(0, ("Got invalid traverse data of length %d\n",
                                  (int)m->datalen));
                        TALLOC_FREE(conn);
-                       return NT_STATUS_UNEXPECTED_IO_ERROR;
+                       return EIO;
                }
 
                key.dsize = d->keylen;
@@ -1153,14 +1127,14 @@ NTSTATUS ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
                if (key.dsize == 0 && data.dsize == 0) {
                        /* end of traverse */
                        TALLOC_FREE(conn);
-                       return NT_STATUS_OK;
+                       return 0;
                }
 
                if (data.dsize < sizeof(struct ctdb_ltdb_header)) {
                        DEBUG(0, ("Got invalid ltdb header length %d\n",
                                  (int)data.dsize));
                        TALLOC_FREE(conn);
-                       return NT_STATUS_UNEXPECTED_IO_ERROR;
+                       return EIO;
                }
                data.dsize -= sizeof(struct ctdb_ltdb_header);
                data.dptr += sizeof(struct ctdb_ltdb_header);
@@ -1169,7 +1143,7 @@ NTSTATUS ctdbd_traverse(struct ctdbd_connection *master, uint32_t db_id,
                        fn(key, data, private_data);
                }
        }
-       return NT_STATUS_OK;
+       return 0;
 }
 
 /*
@@ -1203,18 +1177,18 @@ static void smbd_ctdb_canonicalize_ip(const struct sockaddr_storage *in,
  * Register us as a server for a particular tcp connection
  */
 
-NTSTATUS ctdbd_register_ips(struct ctdbd_connection *conn,
-                           const struct sockaddr_storage *_server,
-                           const struct sockaddr_storage *_client,
-                           int (*cb)(uint32_t src_vnn, uint32_t dst_vnn,
-                                     uint64_t dst_srvid,
-                                     const uint8_t *msg, size_t msglen,
-                                     void *private_data),
-                           void *private_data)
+int ctdbd_register_ips(struct ctdbd_connection *conn,
+                      const struct sockaddr_storage *_server,
+                      const struct sockaddr_storage *_client,
+                      int (*cb)(uint32_t src_vnn, uint32_t dst_vnn,
+                                uint64_t dst_srvid,
+                                const uint8_t *msg, size_t msglen,
+                                void *private_data),
+                      void *private_data)
 {
        struct ctdb_control_tcp_addr p;
        TDB_DATA data = { .dptr = (uint8_t *)&p, .dsize = sizeof(p) };
-       NTSTATUS status;
+       int ret;
        struct sockaddr_storage client;
        struct sockaddr_storage server;
 
@@ -1235,17 +1209,17 @@ NTSTATUS ctdbd_register_ips(struct ctdbd_connection *conn,
                memcpy(&p.src.ip6, &client, sizeof(p.src.ip6));
                break;
        default:
-               return NT_STATUS_INTERNAL_ERROR;
+               return EIO;
        }
 
        /*
         * We want to be told about IP releases
         */
 
-       status = register_with_ctdbd(conn, CTDB_SRVID_RELEASE_IP,
-                                    cb, private_data);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
+       ret = register_with_ctdbd(conn, CTDB_SRVID_RELEASE_IP,
+                                 cb, private_data);
+       if (ret != 0) {
+               return ret;
        }
 
        /*
@@ -1253,81 +1227,88 @@ NTSTATUS ctdbd_register_ips(struct ctdbd_connection *conn,
         * can send an extra ack to trigger a reset for our client, so it
         * immediately reconnects
         */
-       return ctdbd_control(conn, CTDB_CURRENT_NODE,
-                            CTDB_CONTROL_TCP_CLIENT, 0,
-                            CTDB_CTRL_FLAG_NOREPLY, data, NULL, NULL, NULL);
+       ret = ctdbd_control(conn, CTDB_CURRENT_NODE,
+                           CTDB_CONTROL_TCP_CLIENT, 0,
+                           CTDB_CTRL_FLAG_NOREPLY, data, NULL, NULL,
+                           NULL);
+       if (ret != 0) {
+               return ret;
+       }
+       return 0;
 }
 
 /*
   call a control on the local node
  */
-NTSTATUS ctdbd_control_local(struct ctdbd_connection *conn, uint32_t opcode,
-                            uint64_t srvid, uint32_t flags, TDB_DATA data,
-                            TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
-                            int *cstatus)
+int ctdbd_control_local(struct ctdbd_connection *conn, uint32_t opcode,
+                       uint64_t srvid, uint32_t flags, TDB_DATA data,
+                       TALLOC_CTX *mem_ctx, TDB_DATA *outdata,
+                       int *cstatus)
 {
-       return ctdbd_control(conn, CTDB_CURRENT_NODE, opcode, srvid, flags, data, mem_ctx, outdata, cstatus);
+       return ctdbd_control(conn, CTDB_CURRENT_NODE, opcode, srvid, flags, data,
+                            mem_ctx, outdata, cstatus);
 }
 
-NTSTATUS ctdb_watch_us(struct ctdbd_connection *conn)
+int ctdb_watch_us(struct ctdbd_connection *conn)
 {
-       struct ctdb_client_notify_register reg_data;
+       struct ctdb_notify_data_old reg_data;
        size_t struct_len;
-       NTSTATUS status;
+       int ret;
        int cstatus;
 
        reg_data.srvid = CTDB_SRVID_SAMBA_NOTIFY;
        reg_data.len = 1;
        reg_data.notify_data[0] = 0;
 
-       struct_len = offsetof(struct ctdb_client_notify_register,
+       struct_len = offsetof(struct ctdb_notify_data_old,
                              notify_data) + reg_data.len;
 
-       status = ctdbd_control_local(
+       ret = ctdbd_control_local(
                conn, CTDB_CONTROL_REGISTER_NOTIFY, conn->rand_srvid, 0,
                make_tdb_data((uint8_t *)&reg_data, struct_len),
                NULL, NULL, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
+       if (ret != 0) {
                DEBUG(1, ("ctdbd_control_local failed: %s\n",
-                         nt_errstr(status)));
+                         strerror(ret)));
        }
-       return status;
+       return ret;
 }
 
-NTSTATUS ctdb_unwatch(struct ctdbd_connection *conn)
+int ctdb_unwatch(struct ctdbd_connection *conn)
 {
        struct ctdb_client_notify_deregister dereg_data;
-       NTSTATUS status;
+       int ret;
        int cstatus;
 
        dereg_data.srvid = CTDB_SRVID_SAMBA_NOTIFY;
 
-       status = ctdbd_control_local(
+       ret = ctdbd_control_local(
                conn, CTDB_CONTROL_DEREGISTER_NOTIFY, conn->rand_srvid, 0,
                make_tdb_data((uint8_t *)&dereg_data, sizeof(dereg_data)),
                NULL, NULL, &cstatus);
-       if (!NT_STATUS_IS_OK(status)) {
+       if (ret != 0) {
                DEBUG(1, ("ctdbd_control_local failed: %s\n",
-                         nt_errstr(status)));
+                         strerror(ret)));
        }
-       return status;
+       return ret;
 }
 
-NTSTATUS ctdbd_probe(void)
+int ctdbd_probe(const char *sockname, int timeout)
 {
        /*
         * Do a very early check if ctdbd is around to avoid an abort and core
         * later
         */
        struct ctdbd_connection *conn = NULL;
-       NTSTATUS status;
+       int ret;
 
-       status = ctdbd_messaging_connection(talloc_tos(), &conn);
+       ret = ctdbd_messaging_connection(talloc_tos(), sockname, timeout,
+                                        &conn);
 
        /*
         * We only care if we can connect.
         */
        TALLOC_FREE(conn);
 
-       return status;
+       return ret;
 }