From: José Bollo <jose.bollo@iot.bzh>
Date: Fri, 4 May 2018 10:12:30 +0000 (+0200)
Subject: afb-hook: Factorize call to pattern matching
X-Git-Tag: flounder_5.99.1~24
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=ee625f6a8a4b8261fc5a2abf491d7ee490078134;p=src%2Fapp-framework-binder.git

afb-hook: Factorize call to pattern matching

The pattern matching actually use fnmatch but this
could be changed in the futur to some lighter version.

Change-Id: I4f3617cd4a5bdf95988428cb184cef9e9543771d
Signed-off-by: José Bollo <jose.bollo@iot.bzh>
---

diff --git a/src/afb-hook.c b/src/afb-hook.c
index 83ffb5f8..f5868b18 100644
--- a/src/afb-hook.c
+++ b/src/afb-hook.c
@@ -23,7 +23,6 @@
 #include <string.h>
 #include <pthread.h>
 #include <unistd.h>
-#include <fnmatch.h>
 #include <sys/uio.h>
 
 #include <json-c/json.h>
@@ -41,6 +40,9 @@
 #include "afb-api.h"
 #include "verbose.h"
 
+#include <fnmatch.h>
+#define MATCH(pattern,string)   (!fnmatch((pattern),(string),FNM_CASEFOLD|FNM_EXTMATCH))
+
 /**
  * Definition of a hook for xreq
  */
@@ -1307,7 +1309,7 @@ static struct afb_hook_evt_itf hook_evt_default_itf = {
 	while (hook) { \
 		if (hook->itf->hook_evt_##what \
 		 && (hook->flags & afb_hook_flag_evt_##what) != 0 \
-		 && (!hook->pattern || !fnmatch(hook->pattern, evt, FNM_CASEFOLD))) { \
+		 && (!hook->pattern || MATCH(hook->pattern, evt))) { \
 			hook->itf->hook_evt_##what(hook->closure, &hookid, __VA_ARGS__); \
 		} \
 		hook = hook->next; \
@@ -1370,7 +1372,7 @@ int afb_hook_flags_evt(const char *name)
 		pthread_rwlock_rdlock(&rwlock);
 		hook = list_of_evt_hooks;
 		while (hook) {
-			if (!name || !hook->pattern || !fnmatch(hook->pattern, name, FNM_CASEFOLD))
+			if (!name || !hook->pattern || MATCH(hook->pattern, name))
 				flags |= hook->flags;
 			hook = hook->next;
 		}
@@ -1509,7 +1511,7 @@ static struct afb_hook_session_itf hook_session_default_itf = {
 	while (hook) { \
 		if (hook->itf->hook_session_##what \
 		 && (hook->flags & afb_hook_flag_session_##what) != 0 \
-		 && (!hook->pattern || !fnmatch(hook->pattern, (sessid?:(sessid=afb_session_uuid(session))), FNM_CASEFOLD))) { \
+		 && (!hook->pattern || MATCH(hook->pattern, (sessid?:(sessid=afb_session_uuid(session)))))) { \
 			hook->itf->hook_session_##what(hook->closure, &hookid, __VA_ARGS__); \
 		} \
 		hook = hook->next; \