X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fafb-apis.c;h=9ffffdb3b5d22ed632f419c7e7df20bef83e09b4;hb=3bf52bb36ed428d0a7b947519fbccc7c376fd4a9;hp=834850f222490f7743a706a13d3bf8957227cc3a;hpb=7ea1657b459aea2cc6ef9332621a19d7e2676b1d;p=src%2Fapp-framework-binder.git

diff --git a/src/afb-apis.c b/src/afb-apis.c
index 834850f2..9ffffdb3 100644
--- a/src/afb-apis.c
+++ b/src/afb-apis.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2016 "IoT.bzh"
+ * Copyright (C) 2016, 2017 "IoT.bzh"
  * Author "Fulup Ar Foll"
  * Author José Bollo <jose.bollo@iot.bzh>
  *
@@ -21,60 +21,113 @@
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
+#include <errno.h>
 
-#include "session.h"
+#include "afb-session.h"
 #include "verbose.h"
 #include "afb-apis.h"
 #include "afb-context.h"
-#include "afb-req-itf.h"
+#include "afb-hook.h"
+#include <afb/afb-req-itf.h>
 
 struct api_desc {
-	struct afb_api api;
 	const char *name;
-	size_t namelen;
+	struct afb_api api;
 };
 
 static struct api_desc *apis_array = NULL;
 static int apis_count = 0;
 
-int afb_apis_count()
+/**
+ * Checks wether 'name' is a valid API name.
+ * @return 1 if valid, 0 otherwise
+ */
+int afb_apis_is_valid_api_name(const char *name)
 {
-	return apis_count;
+	unsigned char c;
+
+	c = (unsigned char)*name;
+	if (c == 0)
+		/* empty names aren't valid */
+		return 0;
+
+	do {
+		if (c < (unsigned char)'\x80') {
+			switch(c) {
+			default:
+				if (c > ' ')
+					break;
+			case '"':
+			case '#':
+			case '%':
+			case '&':
+			case '\'':
+			case '/':
+			case '?':
+			case '`':
+			case '\\':
+			case '\x7f':
+				return 0;
+			}
+		}
+		c = (unsigned char)*++name;
+	} while(c != 0);
+	return 1;
 }
 
+/**
+ * Adds the api of 'name' described by 'api'.
+ * @param name the name of the api to add (have to survive, not copied!)
+ * @param api the api
+ * @returns 0 in case of success or -1 in case
+ * of error with errno set:
+ *   - EINVAL if name isn't valid
+ *   - EEXIST if name already registered
+ *   - ENOMEM when out of memory
+ */
 int afb_apis_add(const char *name, struct afb_api api)
 {
 	struct api_desc *apis;
-	size_t len;
-	int i;
+	int i, c;
 
-	/* check existing or not */
-	len = strlen(name);
-	if (len == 0) {
-		ERROR("empty api name forbidden");
+	/* Checks the api name */
+	if (!afb_apis_is_valid_api_name(name)) {
+		ERROR("invalid api name forbidden (name is '%s')", name);
+		errno = EINVAL;
 		goto error;
 	}
 
 	/* check previously existing plugin */
 	for (i = 0 ; i < apis_count ; i++) {
-		if (!strcasecmp(apis_array[i].name, name)) {
+		c = strcasecmp(apis_array[i].name, name);
+		if (c == 0) {
 			ERROR("api of name %s already exists", name);
+			errno = EEXIST;
 			goto error;
 		}
+		if (c > 0)
+			break;
 	}
 
 	/* allocates enough memory */
 	apis = realloc(apis_array, ((unsigned)apis_count + 1) * sizeof * apis);
 	if (apis == NULL) {
 		ERROR("out of memory");
+		errno = ENOMEM;
 		goto error;
 	}
 	apis_array = apis;
 
+	/* copy higher part of the array */
+	c = apis_count;
+	while (c > i) {
+		apis_array[c] = apis_array[c - 1];
+		c--;
+	}
+
 	/* record the plugin */
-	apis = &apis_array[apis_count];
+	apis = &apis_array[i];
 	apis->api = api;
-	apis->namelen = len;
 	apis->name = name;
 	apis_count++;
 
@@ -84,24 +137,72 @@ error:
 	return -1;
 }
 
-void afb_apis_call_(struct afb_req req, struct afb_context *context, const char *api, const char *verb)
+/**
+ * Dispatch the request 'req' with the 'context' to the
+ * method of 'api' and 'verb'.
+ * @param req the request to dispatch
+ * @param context the context of the request
+ * @param api the api of the verb
+ * @param verb the verb within the api
+ */
+void afb_apis_call(struct afb_req req, struct afb_context *context, const char *api, const char *verb)
 {
-	afb_apis_call(req, context, api, strlen(api), verb, strlen(verb));
+	int i, c, up, lo;
+	const struct api_desc *a;
+
+	/* init hooking the request */
+	req = afb_hook_req_call(req, context, api, verb);
+
+	/* dichotomic search of the api */
+	/* initial slice */
+	lo = 0;
+	up = apis_count;
+	for (;;) {
+		/* check remaining slice */
+		if (lo >= up) {
+			/* empty ?! */
+			afb_req_fail(req, "fail", "api not found");
+			break;
+		}
+		/* check the mid of the slice */
+		i = (lo + up) >> 1;
+		a = &apis_array[i];
+		c = strcasecmp(a->name, api);
+		if (c == 0) {
+			/* api found */
+			context->api_key = a->api.closure;
+			a->api.call(a->api.closure, req, context, verb);
+			break;
+		}
+		/* update the slice */
+		if (c < 0)
+			lo = i + 1;
+		else
+			up = i;
+	}
 }
 
-void afb_apis_call(struct afb_req req, struct afb_context *context, const char *api, size_t lenapi, const char *verb, size_t lenverb)
+int afb_apis_start_service(const char *api, int share_session, int onneed)
 {
 	int i;
-	const struct api_desc *a;
 
-	a = apis_array;
-	for (i = 0 ; i < apis_count ; i++, a++) {
-		if (a->namelen == lenapi && !strncasecmp(a->name, api, lenapi)) {
-			context->api_index = i;
-			a->api.call(a->api.closure, req, context, verb, lenverb);
-			return;
-		}
+	for (i = 0 ; i < apis_count ; i++) {
+		if (!strcasecmp(apis_array[i].name, api))
+			return apis_array[i].api.service_start(apis_array[i].api.closure, share_session, onneed);
+	}
+	ERROR("can't find service %s", api);
+	return -1;
+}
+
+int afb_apis_start_all_services(int share_session)
+{
+	int i, rc;
+
+	for (i = 0 ; i < apis_count ; i++) {
+		rc = apis_array[i].api.service_start(apis_array[i].api.closure, share_session, 1);
+		if (rc < 0)
+			return rc;
 	}
-	afb_req_fail(req, "fail", "api not found");
+	return 0;
 }