Add pattern and exclude matching for builtin plugins

[connman] / src / plugin.c

diff --git a/src/plugin.c b/src/plugin.c
index 1b57365..9af616f 100644 (file)
--- a/src/plugin.c
+++ b/src/plugin.c
@@ -27,6 +27,10 @@
 
 #include <glib.h>
 
+#ifdef CONNMAN_PLUGIN_BUILTIN
+#undef CONNMAN_PLUGIN_BUILTIN
+#endif
+
 #include "connman.h"
 
 static GSList *plugins = NULL;
@@ -70,15 +74,44 @@ static gboolean add_plugin(void *handle, struct connman_plugin_desc *desc)
        return TRUE;
 }
 
+#include "builtin.h"
+
+static gboolean check_plugin(struct connman_plugin_desc *desc,
+                               const char *pattern, const char *exclude)
+{
+       if (exclude != NULL &&
+                       g_pattern_match_simple(exclude, desc->name) == TRUE) {
+               DBG("excluding %s", desc->description);
+               return FALSE;
+       }
+
+       if (pattern != NULL &&
+                       g_pattern_match_simple(pattern, desc->name) == FALSE) {
+               DBG("ignoring %s", desc->description);
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
 int __connman_plugin_init(const char *pattern, const char *exclude)
 {
        GSList *list;
        GDir *dir;
        const gchar *file;
        gchar *filename;
+       unsigned int i;
 
        DBG("");
 
+       for (i = 0; __connman_builtin[i]; i++) {
+               if (check_plugin(__connman_builtin[i],
+                                               pattern, exclude) == FALSE)
+                       continue;
+
+               add_plugin(NULL, __connman_builtin[i]);
+       }
+
        dir = g_dir_open(PLUGINDIR, 0, NULL);
        if (dir != NULL) {
                while ((file = g_dir_read_name(dir)) != NULL) {
@@ -108,16 +141,7 @@ int __connman_plugin_init(const char *pattern, const char *exclude)
                                continue;
                        }
 
-                       if (exclude != NULL && g_pattern_match_simple(exclude,
-                                                       desc->name) == TRUE) {
-                               DBG("excluding %s", desc->description);
-                               dlclose(handle);
-                               continue;
-                       }
-
-                       if (pattern != NULL && g_pattern_match_simple(pattern,
-                                                       desc->name) == FALSE) {
-                               DBG("ignoring %s", desc->description);
+                       if (check_plugin(desc, pattern, exclude) == FALSE) {
                                dlclose(handle);
                                continue;
                        }
@@ -153,7 +177,8 @@ void __connman_plugin_cleanup(void)
                if (plugin->active == TRUE && plugin->desc->exit)
                        plugin->desc->exit();
 
-               dlclose(plugin->handle);
+               if (plugin->handle != NULL)
+                       dlclose(plugin->handle);
 
                g_free(plugin);
        }