Simplify interfaces initialization

Make as much as possible static, and remove use of talloc_reference and
allocation/deallocation of memory when not necessary.
Fix also responder use of rctx->conn, was mistakenly used for both
monitor and dp connections.

1 files changed, 0 insertions, 35 deletions

diff --git a/server/monitor/monitor_sbus.c b/server/monitor/monitor_sbus.c
index 5d49f003d..817b42ae6 100644
--- a/server/monitor/monitor_sbus.c
+++ b/server/monitor/monitor_sbus.c
@@ -54,38 +54,3 @@ done:
     return ret;
 }
 
-int monitor_init_sbus_methods(TALLOC_CTX *mem_ctx,
-                              struct sbus_method *methods,
-                              struct sbus_method_ctx **sm_ctx)
-{
-    struct sbus_method_ctx *method_ctx;
-    int ret;
-
-    method_ctx = talloc_zero(mem_ctx, struct sbus_method_ctx);
-    if (!method_ctx) {
-        ret = ENOMEM;
-        goto fail;
-    }
-
-    method_ctx->interface = talloc_strdup(method_ctx, SERVICE_INTERFACE);
-    if (method_ctx->interface == NULL) {
-        ret = ENOMEM;
-        goto fail;
-    }
-
-    method_ctx->path = talloc_strdup(method_ctx, SERVICE_PATH);
-    if (method_ctx->path == NULL) {
-        ret = ENOMEM;
-        goto fail;
-    }
-
-    method_ctx->methods = methods;
-    method_ctx->message_handler = sbus_message_handler;
-
-    *sm_ctx = method_ctx;
-    return EOK;
-
-fail:
-    talloc_free(method_ctx);
-    return ret;
-}