X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fafb-hreq.c;h=3a47766f61e3ca1a3f0a689b24e73c8c2096d1ab;hb=44110f28e2502ad2fb5676b80ca420db470870ff;hp=9c6309123fd891d33f38b64e0b7bff69ae49d685;hpb=c6380108964e71f533d8c672bb9c217020a95e8d;p=src%2Fapp-framework-binder.git

diff --git a/src/afb-hreq.c b/src/afb-hreq.c
index 9c630912..3a47766f 100644
--- a/src/afb-hreq.c
+++ b/src/afb-hreq.c
@@ -33,7 +33,6 @@
 #endif
 
 #include "afb-method.h"
-#include <afb/afb-req-common.h>
 #include "afb-msg-json.h"
 #include "afb-context.h"
 #include "afb-hreq.h"
@@ -272,22 +271,38 @@ static const char *mimetype_fd_name(int fd, const char *filename)
 	const char *extension = strrchr(filename, '.');
 	if (extension) {
 		static const char *const known[][2] = {
-			{ ".js",   "text/javascript" },
-			{ ".html", "text/html" },
-			{ ".css",  "text/css" },
-			{ ".ico",  "image/x-icon"},
-			{ ".png",  "image/png" },
-			{ ".svg",  "image/svg+xml" },
-			{ ".ttf",  "application/x-font-ttf"},
-			{ NULL, NULL }
+			/* keep it sorted for dichotomic search */
+			{ ".css",	"text/css" },
+			{ ".gif",	"image/gif" },
+			{ ".html",	"text/html" },
+			{ ".htm",	"text/html" },
+			{ ".ico",	"image/x-icon"},
+			{ ".jpeg",	"image/jpeg" },
+			{ ".jpg",	"image/jpeg" },
+			{ ".js",	"text/javascript" },
+			{ ".json",	"application/json" },
+			{ ".mp3",	"audio/mpeg" },
+			{ ".png",	"image/png" },
+			{ ".svg",	"image/svg+xml" },
+			{ ".ttf",	"application/x-font-ttf"},
+			{ ".txt",	"text/plain" },
+			{ ".wav",	"audio/x-wav" },
+			{ ".xht",	"application/xhtml+xml" },
+			{ ".xhtml",	"application/xhtml+xml" },
+			{ ".xml",	"application/xml" }
 		};
-		int i = 0;
-		while (known[i][0]) {
-			if (!strcasecmp(extension, known[i][0])) {
+		int i, c, l = 0, u = sizeof known / sizeof *known;
+		while (l < u) {
+			i = (l + u) >> 1;
+			c = strcasecmp(extension, known[i][0]);
+			if (!c) {
 				result = known[i][1];
 				break;
 			}
-			i++;
+			if (c < 0)
+				u = i;
+			else
+				l = i + 1;
 		}
 	}
 #endif
@@ -317,20 +332,22 @@ static void req_destroy(struct afb_xreq *xreq)
 	}
 	afb_context_disconnect(&hreq->xreq.context);
 	json_object_put(hreq->json);
-	free((char*)hreq->xreq.api);
-	free((char*)hreq->xreq.verb);
+	free((char*)hreq->xreq.request.api);
+	free((char*)hreq->xreq.request.verb);
 	afb_cred_unref(hreq->xreq.cred);
 	free(hreq);
 }
 
 void afb_hreq_addref(struct afb_hreq *hreq)
 {
-	afb_xreq_addref(&hreq->xreq);
+	afb_xreq_unhooked_addref(&hreq->xreq);
 }
 
 void afb_hreq_unref(struct afb_hreq *hreq)
 {
-	afb_xreq_unref(&hreq->xreq);
+	if (hreq->replied)
+		hreq->xreq.replied = 1;
+	afb_xreq_unhooked_unref(&hreq->xreq);
 }
 
 /*
@@ -913,15 +930,15 @@ static void req_success(struct afb_xreq *xreq, json_object *obj, const char *inf
 
 void afb_hreq_call(struct afb_hreq *hreq, struct afb_apiset *apiset, const char *api, size_t lenapi, const char *verb, size_t lenverb)
 {
-	hreq->xreq.api = strndup(api, lenapi);
-	hreq->xreq.verb = strndup(verb, lenverb);
-	if (hreq->xreq.api == NULL || hreq->xreq.verb == NULL) {
+	hreq->xreq.request.api = strndup(api, lenapi);
+	hreq->xreq.request.verb = strndup(verb, lenverb);
+	if (hreq->xreq.request.api == NULL || hreq->xreq.request.verb == NULL) {
 		ERROR("Out of memory");
 		afb_hreq_reply_error(hreq, MHD_HTTP_INTERNAL_SERVER_ERROR);
 	} else if (afb_hreq_init_context(hreq) < 0) {
 		afb_hreq_reply_error(hreq, MHD_HTTP_INTERNAL_SERVER_ERROR);
 	} else {
-		afb_xreq_addref(&hreq->xreq); /* TODO check if needed */
+		afb_xreq_unhooked_addref(&hreq->xreq);
 		afb_xreq_process(&hreq->xreq, apiset);
 	}
 }