winbindd: Do not use group_list->out.resume_index after free
[samba.git] / source4 / winbind / wb_irpc.c
index 2f2b0780c7b1a18fcd0b58b7fdc9ee2f81be9f41..7a4ca69baa840f2ad7ca9a9858f8be4a9d30e90c 100644 (file)
@@ -30,7 +30,7 @@ struct wb_irpc_SamLogon_state {
        struct winbind_SamLogon *req;
 };
 
-static void wb_irpc_SamLogon_callback(struct composite_context *ctx);
+static void wb_irpc_SamLogon_callback(struct tevent_req *subreq);
 
 static NTSTATUS wb_irpc_SamLogon(struct irpc_message *msg, 
                                 struct winbind_SamLogon *req)
@@ -38,7 +38,7 @@ static NTSTATUS wb_irpc_SamLogon(struct irpc_message *msg,
        struct wbsrv_service *service = talloc_get_type(msg->private_data,
                                        struct wbsrv_service);
        struct wb_irpc_SamLogon_state *s;
-       struct composite_context *ctx;
+       struct tevent_req *subreq;
 
        DEBUG(5, ("wb_irpc_SamLogon called\n"));
 
@@ -48,25 +48,28 @@ static NTSTATUS wb_irpc_SamLogon(struct irpc_message *msg,
        s->msg = msg;
        s->req = req;
 
-       ctx = wb_sam_logon_send(msg, service, req);
-       NT_STATUS_HAVE_NO_MEMORY(ctx);
+       subreq = wb_sam_logon_send(s,
+                                  service->task->event_ctx,
+                                  service, req);
+       NT_STATUS_HAVE_NO_MEMORY(subreq);
 
-       ctx->async.fn = wb_irpc_SamLogon_callback;
-       ctx->async.private_data = s;
+       tevent_req_set_callback(subreq, wb_irpc_SamLogon_callback, s);
 
        msg->defer_reply = true;
        return NT_STATUS_OK;
 }
 
-static void wb_irpc_SamLogon_callback(struct composite_context *ctx)
+static void wb_irpc_SamLogon_callback(struct tevent_req *subreq)
 {
-       struct wb_irpc_SamLogon_state *s = talloc_get_type(ctx->async.private_data,
-                                          struct wb_irpc_SamLogon_state);
+       struct wb_irpc_SamLogon_state *s =
+               tevent_req_callback_data(subreq,
+               struct wb_irpc_SamLogon_state);
        NTSTATUS status;
 
        DEBUG(5, ("wb_irpc_SamLogon_callback called\n"));
 
-       status = wb_sam_logon_recv(ctx, s, s->req);
+       status = wb_sam_logon_recv(subreq, s, s->req);
+       TALLOC_FREE(subreq);
 
        irpc_send_reply(s->msg, status);
 }
@@ -76,7 +79,7 @@ struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state {
        struct winbind_DsrUpdateReadOnlyServerDnsRecords *req;
 };
 
-static void wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback(struct composite_context *ctx);
+static void wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback(struct tevent_req *subreq);
 
 static NTSTATUS wb_irpc_DsrUpdateReadOnlyServerDnsRecords(struct irpc_message *msg,
                                 struct winbind_DsrUpdateReadOnlyServerDnsRecords *req)
@@ -84,7 +87,7 @@ static NTSTATUS wb_irpc_DsrUpdateReadOnlyServerDnsRecords(struct irpc_message *m
        struct wbsrv_service *service = talloc_get_type(msg->private_data,
                                        struct wbsrv_service);
        struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state *s;
-       struct composite_context *ctx;
+       struct tevent_req *subreq;
 
        DEBUG(5, ("wb_irpc_DsrUpdateReadOnlyServerDnsRecords called\n"));
 
@@ -94,93 +97,30 @@ static NTSTATUS wb_irpc_DsrUpdateReadOnlyServerDnsRecords(struct irpc_message *m
        s->msg = msg;
        s->req = req;
 
-       ctx = wb_update_rodc_dns_send(msg, service, req);
-       NT_STATUS_HAVE_NO_MEMORY(ctx);
+       subreq = wb_update_rodc_dns_send(s,
+                                        service->task->event_ctx,
+                                        service, req);
+       NT_STATUS_HAVE_NO_MEMORY(subreq);
 
-       ctx->async.fn = wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback;
-       ctx->async.private_data = s;
+       tevent_req_set_callback(subreq,
+                               wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback,
+                               s);
 
        msg->defer_reply = true;
        return NT_STATUS_OK;
 }
 
-static void wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback(struct composite_context *ctx)
+static void wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback(struct tevent_req *subreq)
 {
-       struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state *s = talloc_get_type(ctx->async.private_data,
-                                          struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state);
+       struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state *s =
+               tevent_req_callback_data(subreq,
+               struct wb_irpc_DsrUpdateReadOnlyServerDnsRecords_state);
        NTSTATUS status;
 
        DEBUG(5, ("wb_irpc_DsrUpdateReadOnlyServerDnsRecords_callback called\n"));
 
-       status = wb_update_rodc_dns_recv(ctx, s, s->req);
-
-       irpc_send_reply(s->msg, status);
-}
-
-struct wb_irpc_get_idmap_state {
-       struct irpc_message *msg;
-       struct winbind_get_idmap *req;
-       int level;
-};
-
-static void wb_irpc_get_idmap_callback(struct composite_context *ctx);
-
-static NTSTATUS wb_irpc_get_idmap(struct irpc_message *msg,
-                                 struct winbind_get_idmap *req)
-{
-       struct wbsrv_service *service = talloc_get_type(msg->private_data,
-                                       struct wbsrv_service);
-       struct wb_irpc_get_idmap_state *s;
-       struct composite_context *ctx = NULL;
-
-       DEBUG(5, ("wb_irpc_get_idmap called\n"));
-
-       s = talloc(msg, struct wb_irpc_get_idmap_state);
-       NT_STATUS_HAVE_NO_MEMORY(s);
-
-       s->msg = msg;
-       s->req = req;
-       s->level = req->in.level;
-
-       switch(s->level) {
-               case WINBIND_IDMAP_LEVEL_SIDS_TO_XIDS:
-                       ctx = wb_sids2xids_send(msg, service, req->in.count,
-                                               req->in.ids);
-                       break;
-               case WINBIND_IDMAP_LEVEL_XIDS_TO_SIDS:
-                       ctx = wb_xids2sids_send(msg, service, req->in.count,
-                                               req->in.ids);
-                       break;
-       }
-       NT_STATUS_HAVE_NO_MEMORY(ctx);
-
-       composite_continue(ctx, ctx, wb_irpc_get_idmap_callback, s);
-       msg->defer_reply = true;
-
-       return NT_STATUS_OK;
-}
-
-static void wb_irpc_get_idmap_callback(struct composite_context *ctx)
-{
-       struct wb_irpc_get_idmap_state *s;
-       NTSTATUS status;
-
-       DEBUG(5, ("wb_irpc_get_idmap_callback called\n"));
-
-       s = talloc_get_type(ctx->async.private_data,
-                           struct wb_irpc_get_idmap_state);
-
-       switch(s->level) {
-               case WINBIND_IDMAP_LEVEL_SIDS_TO_XIDS:
-                       status = wb_sids2xids_recv(ctx, &s->req->out.ids, NULL);
-                       break;
-               case WINBIND_IDMAP_LEVEL_XIDS_TO_SIDS:
-                       status = wb_xids2sids_recv(ctx, &s->req->out.ids);
-                       break;
-               default:
-                       status = NT_STATUS_INTERNAL_ERROR;
-                       break;
-       }
+       status = wb_update_rodc_dns_recv(subreq, s, s->req);
+       TALLOC_FREE(subreq);
 
        irpc_send_reply(s->msg, status);
 }
@@ -199,9 +139,5 @@ NTSTATUS wbsrv_init_irpc(struct wbsrv_service *service)
                               wb_irpc_DsrUpdateReadOnlyServerDnsRecords, service);
        NT_STATUS_NOT_OK_RETURN(status);
 
-       status = IRPC_REGISTER(service->task->msg_ctx, winbind, WINBIND_GET_IDMAP,
-                              wb_irpc_get_idmap, service);
-       NT_STATUS_NOT_OK_RETURN(status);
-
        return NT_STATUS_OK;
 }