X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fafb-subcall.c;h=7026f20e956f3b35cc253eda66c339e9a67c5a61;hb=4846cec0be7f41f992b9f89b7adb97033e15b1a0;hp=82128d8a4fee60a35a39163c9f687bf8f6822fad;hpb=5e7d680c173ddbb26db4e49f7738f82e89299362;p=src%2Fapp-framework-binder.git

diff --git a/src/afb-subcall.c b/src/afb-subcall.c
index 82128d8a..7026f20e 100644
--- a/src/afb-subcall.c
+++ b/src/afb-subcall.c
@@ -29,6 +29,7 @@
 #include "afb-context.h"
 #include "afb-xreq.h"
 #include "verbose.h"
+#include "jobs.h"
 
 struct subcall;
 
@@ -83,16 +84,6 @@ static int subcall_unsubscribe(void *closure, struct afb_event event)
 	return afb_xreq_unsubscribe(subcall->caller, event);
 }
 
-void afb_subcall_internal_error(void (*callback)(void*, int, struct json_object*), void *closure)
-{
-	static struct json_object *obj;
-
-	if (obj == NULL)
-		obj = afb_msg_json_reply_error("failed", "internal error", NULL, NULL);
-
-	callback(closure, 1, obj);
-}
-
 static struct subcall *create_subcall(struct afb_xreq *caller, const char *api, const char *verb, struct json_object *args, void (*callback)(void*, int, struct json_object*), void *closure)
 {
 	struct subcall *subcall;
@@ -102,7 +93,6 @@ static struct subcall *create_subcall(struct afb_xreq *caller, const char *api,
 		return NULL;
 	}
 
-
 	afb_context_subinit(&subcall->xreq.context, &caller->context);
 	subcall->xreq.refcount = 1;
 	subcall->xreq.json = args;
@@ -113,6 +103,8 @@ static struct subcall *create_subcall(struct afb_xreq *caller, const char *api,
 	subcall->caller = caller;
 	subcall->callback = callback;
 	subcall->closure = closure;
+	afb_xreq_addref(caller);
+	json_object_get(args);
 	return subcall;
 }
 
@@ -129,13 +121,84 @@ void afb_subcall(
 
 	subcall = create_subcall(caller, api, verb, args, callback, closure);
 	if (subcall == NULL) {
-		afb_subcall_internal_error(callback, closure);
+		callback(closure, 1, afb_msg_json_internal_error());
 		return;
 	}
 
-	afb_xreq_addref(caller);
-	afb_apis_xcall(&subcall->xreq);
+	afb_apis_call(&subcall->xreq);
 	afb_xreq_unref(&subcall->xreq);
 }
 
+struct subcall_sync
+{
+	struct afb_xreq *caller;
+	const char *api;
+	const char *verb;
+	struct json_object *args;
+	struct jobloop *jobloop;
+	struct json_object *result;
+	int iserror;
+};
+
+static void subcall_sync_leave(struct subcall_sync *sync)
+{
+	struct jobloop *jobloop = sync->jobloop;
+	sync->jobloop = NULL;
+	if (jobloop)
+		jobs_leave(jobloop);
+}
+
+static void subcall_sync_reply(void *closure, int iserror, struct json_object *obj)
+{
+	struct subcall_sync *sync = closure;
+
+	sync->iserror = iserror;
+	sync->result = obj;
+	json_object_get(obj);
+	subcall_sync_leave(sync);
+}
+
+static void subcall_sync_enter(int signum, void *closure, struct jobloop *jobloop)
+{
+	struct subcall_sync *sync = closure;
+
+	if (!signum) {
+		sync->jobloop = jobloop;
+		afb_xreq_unhooked_subcall(sync->caller, sync->api, sync->verb, sync->args, subcall_sync_reply, sync);
+	} else {
+		sync->result = json_object_get(afb_msg_json_internal_error());
+		sync->iserror = 1;
+		subcall_sync_leave(sync);
+	}
+}
+
+int afb_subcall_sync(
+		struct afb_xreq *caller,
+		const char *api,
+		const char *verb,
+		struct json_object *args,
+		struct json_object **result
+)
+{
+	int rc;
+	struct subcall_sync sync;
+
+	sync.caller = caller;
+	sync.api = api;
+	sync.verb = verb;
+	sync.args = args;
+	sync.jobloop = NULL;
+	sync.result = NULL;
+	sync.iserror = 1;
+
+	rc = jobs_enter(NULL, 0, subcall_sync_enter, &sync);
+	if (rc < 0) {
+		sync.result = json_object_get(afb_msg_json_internal_error());
+		sync.iserror = 1;
+	}
+	rc = !sync.iserror;
+	*result = sync.result;
+	return rc;
+}
+