]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
aco: Minimize use of regex.
authorCorey Farrell <git@cfware.com>
Tue, 12 Dec 2017 18:55:12 +0000 (13:55 -0500)
committerCorey Farrell <git@cfware.com>
Fri, 15 Dec 2017 15:20:51 +0000 (10:20 -0500)
Remove nearly all use of regex from ACO users.  Still remaining:
* app_confbridge has a legitamate use of option name regex.
* ast_sorcery_object_fields_register is implemented with regex, all
  callers use simple prefix based regex.  I haven't decided the best
  way to fix this in both 13/15 and master.

Change-Id: Ib5ed478218d8a661ace4d2eaaea98b59a897974b

17 files changed:
apps/app_agent_pool.c
apps/app_skel.c
apps/confbridge/conf_config_parser.c
channels/chan_motif.c
main/cdr.c
main/cel.c
main/features_config.c
main/named_acl.c
main/stasis.c
main/udptl.c
res/ari/config.c
res/res_hep.c
res/res_parking.c
res/res_pjsip_notify.c
res/res_statsd.c
res/res_xmpp.c
tests/test_config.c

index a637bbe5f8accd0e595e26f110e8c423a7bedb37..ef8b95ec8be074f9277a2dcef6081d9f5e6c20bc 100644 (file)
@@ -457,11 +457,17 @@ struct agents_cfg {
        struct ao2_container *agents;
 };
 
+static const char *agent_type_blacklist[] = {
+       "general",
+       "agents",
+       NULL,
+};
+
 static struct aco_type agent_type = {
        .type = ACO_ITEM,
        .name = "agent-id",
-       .category_match = ACO_BLACKLIST,
-       .category = "^(general|agents)$",
+       .category_match = ACO_BLACKLIST_ARRAY,
+       .category = (const char *)agent_type_blacklist,
        .item_alloc = agent_cfg_alloc,
        .item_find = agent_cfg_find,
        .item_offset = offsetof(struct agents_cfg, agents),
@@ -473,8 +479,8 @@ static struct aco_type *agent_types[] = ACO_TYPES(&agent_type);
 static struct aco_type general_type = {
        .type = ACO_GLOBAL,
        .name = "global",
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 static struct aco_file agents_conf = {
index 12afdc72639fecf64b393db22e854054abd4d64b..20e6abfcc12c77b2ecf2477b149e8d5991f122cd 100644 (file)
@@ -244,8 +244,8 @@ static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "globals",
        .item_offset = offsetof(struct skel_config, global),
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 struct aco_type *global_options[] = ACO_TYPES(&global_option);
@@ -255,18 +255,24 @@ static struct aco_type sound_option = {
        .type = ACO_GLOBAL,
        .name = "sounds",
        .item_offset = offsetof(struct skel_config, global),
-       .category_match = ACO_WHITELIST,
-       .category = "^sounds$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "sounds",
 };
 
 struct aco_type *sound_options[] = ACO_TYPES(&sound_option);
 
+static const char *level_categories[] = {
+       "general",
+       "sounds",
+       NULL,
+};
+
 /*! \brief An aco_type structure to link the everything but the "general" and "sounds" categories to the skel_level type */
 static struct aco_type level_option = {
        .type = ACO_ITEM,
        .name = "level",
-       .category_match = ACO_BLACKLIST,
-       .category = "^(general|sounds)$",
+       .category_match = ACO_BLACKLIST_ARRAY,
+       .category = (const char *)level_categories,
        .item_alloc = skel_level_alloc,
        .item_find = skel_level_find,
        .item_offset = offsetof(struct skel_config, levels),
index 2de1ec238d1c25b63cd60f09227e39427206500f..16d39352666b88c7af67977bd4d099e72e924a5e 100644 (file)
@@ -377,7 +377,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
                                                regardless if this limit is reached or not.
                                        </para></description>
                                </configOption>
-                               <configOption name="^sound_">
+                               <configOption name="sound_">
                                        <synopsis>Override the various conference bridge sound files</synopsis>
                                        <description><para>
                                                All sounds in the conference are customizable using the bridge profile options below.
@@ -592,8 +592,8 @@ static void *bridge_profile_find(struct ao2_container *container, const char *ca
 static struct aco_type bridge_type = {
        .type = ACO_ITEM,
        .name = "bridge_profile",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .matchfield = "type",
        .matchvalue = "bridge",
        .item_alloc = bridge_profile_alloc,
@@ -629,8 +629,8 @@ static void *user_profile_find(struct ao2_container *container, const char *cate
 static struct aco_type user_type = {
        .type = ACO_ITEM,
        .name  = "user_profile",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .matchfield = "type",
        .matchvalue = "user",
        .item_alloc = user_profile_alloc,
@@ -660,8 +660,8 @@ static void *menu_find(struct ao2_container *container, const char *category)
 static struct aco_type menu_type = {
        .type = ACO_ITEM,
        .name = "menu",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .matchfield = "type",
        .matchvalue = "menu",
        .item_alloc = menu_alloc,
@@ -678,8 +678,8 @@ static struct aco_type *user_types[] = ACO_TYPES(&user_type);
 static struct aco_type general_type = {
        .type = ACO_GLOBAL,
        .name = "global",
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 static struct aco_file confbridge_conf = {
@@ -2149,7 +2149,7 @@ int conf_load_config(void)
        aco_option_register(&cfg_info, "record_file", ACO_EXACT, bridge_types, NULL, OPT_CHAR_ARRAY_T, 0, CHARFLDSET(struct bridge_profile, rec_file));
        aco_option_register(&cfg_info, "regcontext", ACO_EXACT, bridge_types, NULL, OPT_CHAR_ARRAY_T, 0, CHARFLDSET(struct bridge_profile, regcontext));
        aco_option_register(&cfg_info, "language", ACO_EXACT, bridge_types, "en", OPT_CHAR_ARRAY_T, 0, CHARFLDSET(struct bridge_profile, language));
-       aco_option_register_custom(&cfg_info, "^sound_", ACO_REGEX, bridge_types, NULL, sound_option_handler, 0);
+       aco_option_register_custom(&cfg_info, "sound_", ACO_PREFIX, bridge_types, NULL, sound_option_handler, 0);
        /* This option should only be used with the CONFBRIDGE dialplan function */
        aco_option_register_custom(&cfg_info, "template", ACO_EXACT, bridge_types, NULL, bridge_template_handler, 0);
 
index 314103765c85f952fb4c05b7a8cd0363f6aa5918..9393807719dc2a11f1af28a8afdaa152be9136ff 100644 (file)
@@ -543,8 +543,8 @@ static int jingle_endpoint_cmp(void *obj, void *arg, int flags)
 static struct aco_type endpoint_option = {
        .type = ACO_ITEM,
        .name = "endpoint",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .item_alloc = jingle_endpoint_alloc,
        .item_find = jingle_endpoint_find,
        .item_offset = offsetof(struct jingle_config, endpoints),
index 60532fe9fce986eecf4c2b8110a5f4d97f130fd2..501589abab0cfee3cf45a04ba1627d1d4048e3b1 100644 (file)
@@ -241,8 +241,29 @@ static struct aco_type general_option = {
        .type = ACO_GLOBAL,
        .name = "general",
        .item_offset = offsetof(struct module_config, general),
-       .category = "^general$",
-       .category_match = ACO_WHITELIST,
+       .category = "general",
+       .category_match = ACO_WHITELIST_EXACT,
+};
+
+/*! Config sections used by existing modules. Do not add to this list. */
+static const char *ignore_categories[] = {
+       "csv",
+       "custom",
+       "manager",
+       "odbc",
+       "pgsql",
+       "radius",
+       "sqlite",
+       "tds",
+       "mysql",
+       NULL,
+};
+
+static struct aco_type ignore_option = {
+       .type = ACO_IGNORE,
+       .name = "modules",
+       .category = (const char*)ignore_categories,
+       .category_match = ACO_WHITELIST_ARRAY,
 };
 
 static void *module_config_alloc(void);
@@ -252,8 +273,7 @@ static void module_config_post_apply(void);
 /*! \brief The file definition */
 static struct aco_file module_file_conf = {
        .filename = "cdr.conf",
-       .skip_category = "(^csv$|^custom$|^manager$|^odbc$|^pgsql$|^radius$|^sqlite$|^tds$|^mysql$)",
-       .types = ACO_TYPES(&general_option),
+       .types = ACO_TYPES(&general_option, &ignore_option),
 };
 
 CONFIG_INFO_CORE("cdr", cfg_info, module_configs, module_config_alloc,
index 0cdf1be00b288dcb060808284adacb456f793708..d46f185cf734ed59b45574f77e48d111a131cc17 100644 (file)
@@ -244,15 +244,28 @@ static struct aco_type general_option = {
        .type = ACO_GLOBAL,
        .name = "general",
        .item_offset = offsetof(struct cel_config, general),
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
+};
+
+/*! Config sections used by existing modules. Do not add to this list. */
+static const char *ignore_categories[] = {
+       "manager",
+       "radius",
+       NULL,
+};
+
+static struct aco_type ignore_option = {
+       .type = ACO_IGNORE,
+       .name = "modules",
+       .category = (const char*)ignore_categories,
+       .category_match = ACO_WHITELIST_ARRAY,
 };
 
 /*! \brief The config file to be processed for the module. */
 static struct aco_file cel_conf = {
        .filename = "cel.conf",                  /*!< The name of the config file */
-       .types = ACO_TYPES(&general_option),     /*!< The mapping object types to be processed */
-       .skip_category = "(^manager$|^radius$)", /*!< Config sections used by existing modules. Do not add to this list. */
+       .types = ACO_TYPES(&general_option, &ignore_option),     /*!< The mapping object types to be processed */
 };
 
 static int cel_pre_apply_config(void);
index 2689687cc5b69642057a27d577a0892f96bd7903..e2d405740f995494e67dea35f9d5ee845a7e566a 100644 (file)
                                        The <replaceable>DYNAMIC_FEATURES</replaceable> is a <literal>#</literal> separated list of
                                        either applicationmap item names or featuregroup names.</para>
                                </description>
-                               <configOption name="^.*$" regex="true">
+                               <configOption name="">
                                        <synopsis>A custom feature to invoke during a bridged call</synopsis>
                                        <description>
                                                <para>Each item listed here is a comma-separated list of parameters that determine
                                        DTMF sequence used to invoke an applicationmap item to be overridden with
                                        a different sequence.</para>
                                </description>
-                               <configOption name="^.*$" regex="true">
+                               <configOption name="">
                                        <synopsis>Applicationmap item to place in the feature group</synopsis>
                                        <description>
                                                <para>Each item here must be a name of an item in the applicationmap. The
@@ -578,24 +578,24 @@ struct features_config {
 static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "globals",
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
        .item_offset = offsetof(struct features_config, global),
 };
 
 static struct aco_type featuremap_option = {
        .type = ACO_GLOBAL,
        .name = "featuremap",
-       .category_match = ACO_WHITELIST,
-       .category = "^featuremap$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "featuremap",
        .item_offset = offsetof(struct features_config, featuremap),
 };
 
 static struct aco_type applicationmap_option = {
        .type = ACO_GLOBAL,
        .name = "applicationmap",
-       .category_match = ACO_WHITELIST,
-       .category = "^applicationmap$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "applicationmap",
        .item_offset = offsetof(struct features_config, applicationmap),
 };
 
@@ -1851,13 +1851,13 @@ static int load_config(void)
        aco_option_register_custom(&cfg_info, "automixmon", ACO_EXACT, featuremap_options,
                        DEFAULT_FEATUREMAP_AUTOMIXMON, featuremap_handler, 0);
 
-       aco_option_register_custom(&cfg_info, "^.*$", ACO_REGEX, applicationmap_options,
+       aco_option_register_custom(&cfg_info, "", ACO_PREFIX, applicationmap_options,
                        "", applicationmap_handler, 0);
 
-       aco_option_register_custom(&cfg_info, "^.*$", ACO_REGEX, featuregroup_options,
+       aco_option_register_custom(&cfg_info, "", ACO_PREFIX, featuregroup_options,
                        "", featuregroup_handler, 0);
 
-       aco_option_register_custom_nodoc(&cfg_info, "^.*$", ACO_REGEX, parkinglot_options,
+       aco_option_register_custom_nodoc(&cfg_info, "", ACO_PREFIX, parkinglot_options,
                        "", unsupported_handler, 0);
 
        if (aco_process_config(&cfg_info, 0) == ACO_PROCESS_ERROR) {
index 3b81c8c382f7046e52a0ca10b0ce163131019d58..47787e921be12355775ba41c5e0dee1915b1f9ad 100644 (file)
@@ -82,8 +82,8 @@ static void *named_acl_find(struct ao2_container *container, const char *cat);
 static struct aco_type named_acl_type = {
        .type = ACO_ITEM,                  /*!< named_acls are items stored in containers, not individual global objects */
        .name = "named_acl",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",           /*!< Match everything but "general" */
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",           /*!< Match everything but "general" */
        .item_alloc = named_acl_alloc,     /*!< A callback to allocate a new named_acl based on category */
        .item_find = named_acl_find,       /*!< A callback to find a named_acl in some container of named_acls */
        .item_offset = offsetof(struct named_acl_config, named_acl_list), /*!< Could leave this out since 0 */
index 63d17dfaf3fb6b7f522ef3de8121da6bff5ad982..d9785ce73adf75944ec1d67ddb52cde019626f40 100644 (file)
@@ -1436,8 +1436,8 @@ static struct aco_type threadpool_option = {
        .type = ACO_GLOBAL,
        .name = "threadpool",
        .item_offset = offsetof(struct stasis_config, threadpool_options),
-       .category = "^threadpool$",
-       .category_match = ACO_WHITELIST,
+       .category = "threadpool",
+       .category_match = ACO_WHITELIST_EXACT,
 };
 
 static struct aco_type *threadpool_options[] = ACO_TYPES(&threadpool_option);
@@ -1447,8 +1447,8 @@ static struct aco_type declined_option = {
        .type = ACO_GLOBAL,
        .name = "declined_message_types",
        .item_offset = offsetof(struct stasis_config, declined_message_types),
-       .category_match = ACO_WHITELIST,
-       .category = "^declined_message_types$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "declined_message_types",
 };
 
 struct aco_type *declined_options[] = ACO_TYPES(&declined_option);
index 44e9eb9bb7e6d02e8a7f42a5c327cd9f263675e1..83989f7388f50bb307cd7e8eb52494d87ab901fa 100644 (file)
@@ -239,9 +239,9 @@ static int udptl_pre_apply_config(void);
 static struct aco_type general_option = {
        .type = ACO_GLOBAL,
        .name = "global",
-       .category_match = ACO_WHITELIST,
+       .category_match = ACO_WHITELIST_EXACT,
        .item_offset = offsetof(struct udptl_config, general),
-       .category = "^general$",
+       .category = "general",
 };
 
 static struct aco_type *general_options[] = ACO_TYPES(&general_option);
index 2d3a80d0bbc218c1bdd9e8ceb811938bddd9b06f..275f41d96398e962b82ce347b6d59ceb4a6ec867 100644 (file)
@@ -39,8 +39,8 @@ static struct aco_type general_option = {
        .type = ACO_GLOBAL,
        .name = "general",
        .item_offset = offsetof(struct ast_ari_conf, general),
-       .category = "^general$",
-       .category_match = ACO_WHITELIST,
+       .category = "general",
+       .category_match = ACO_WHITELIST_EXACT,
 };
 
 static struct aco_type *general_options[] = ACO_TYPES(&general_option);
@@ -156,8 +156,8 @@ static void *user_find(struct ao2_container *tmp_container, const char *cat)
 static struct aco_type user_option = {
        .type = ACO_ITEM,
        .name = "user",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .matchfield = "type",
        .matchvalue = "user",
        .item_alloc = user_alloc,
index f3b65ad2683c8a82c02628c1a204b21addd80408..5abf9018fa2e0192aaa6a5bafcc00ab3305bdb92 100644 (file)
@@ -260,8 +260,8 @@ static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "general",
        .item_offset = offsetof(struct module_config, general),
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 struct aco_type *global_options[] = ACO_TYPES(&global_option);
index 593134b6052b000e51859640fdca3e9d0a6cc0e3..ddf8012454c97fd81495f7cd1bea6128edf0d809 100644 (file)
@@ -291,8 +291,8 @@ static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "globals",
        .item_offset = offsetof(struct parking_config, global),
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 struct aco_type *global_options[] = ACO_TYPES(&global_option);
@@ -300,8 +300,8 @@ struct aco_type *global_options[] = ACO_TYPES(&global_option);
 static struct aco_type parking_lot_type = {
        .type = ACO_ITEM,
        .name = "parking_lot",
-       .category_match = ACO_BLACKLIST,
-       .category = "^(general)$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .item_alloc = parking_lot_cfg_alloc,
        .item_find = named_item_find,
        .item_offset = offsetof(struct parking_config, parking_lots),
index 8258b3857f83af636102f3aa4cc010e898eda6a0..fdc7ecf54651160b74553c47f5dad260265a24fa 100644 (file)
@@ -82,7 +82,7 @@
                                        order; any other header is treated as part of the SIP
                                        request.</para>
                                </description>
-                               <configOption name="^.*$">
+                               <configOption name="">
                                        <synopsis>A key/value pair to add to a NOTIFY request.</synopsis>
                                        <description>
                                                <para>If the key is <literal>Content</literal>,
@@ -234,8 +234,8 @@ static void *notify_cfg_alloc(void)
 static struct aco_type notify_option = {
        .type = ACO_ITEM,
        .name = "notify",
-       .category_match = ACO_BLACKLIST,
-       .category = "^general$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .item_offset = offsetof(struct notify_cfg, notify_options),
        .item_alloc = notify_option_alloc,
        .item_find = notify_option_find
@@ -993,7 +993,7 @@ static int load_module(void)
                return AST_MODULE_LOAD_DECLINE;
        }
 
-       aco_option_register_custom(&notify_cfg, "^.*$", ACO_REGEX, notify_options,
+       aco_option_register_custom(&notify_cfg, "", ACO_PREFIX, notify_options,
                                   "", notify_option_handler, 0);
 
        if (aco_process_config(&notify_cfg, 0)) {
index b8e4d04758b34bb496839ca08884ac2268e704d2..67166e8208144a09b0a50fe3ca0f84a8590de5fd 100644 (file)
@@ -233,8 +233,8 @@ static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "global",
        .item_offset = offsetof(struct conf, global),
-       .category = "^general$",
-       .category_match = ACO_WHITELIST
+       .category = "general",
+       .category_match = ACO_WHITELIST_EXACT,
 };
 
 static struct aco_type *global_options[] = ACO_TYPES(&global_option);
index 1b0ae427bc7abd3c3eb70a2d82eb9db2bdd5690c..41f89961ce63dd581cdee4ae69971aa77b9156b4 100644 (file)
@@ -822,8 +822,8 @@ static struct aco_type global_option = {
        .type = ACO_GLOBAL,
        .name = "global",
        .item_offset = offsetof(struct xmpp_config, global),
-       .category_match = ACO_WHITELIST,
-       .category = "^general$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "general",
 };
 
 struct aco_type *global_options[] = ACO_TYPES(&global_option);
@@ -831,8 +831,8 @@ struct aco_type *global_options[] = ACO_TYPES(&global_option);
 static struct aco_type client_option = {
        .type = ACO_ITEM,
        .name = "client",
-       .category_match = ACO_BLACKLIST,
-       .category = "^(general)$",
+       .category_match = ACO_BLACKLIST_EXACT,
+       .category = "general",
        .item_alloc = ast_xmpp_client_config_alloc,
        .item_find = xmpp_config_find,
        .item_prelink = xmpp_config_prelink,
index c58bdc70d854b6da13e35531b0037dc177bbe2f3..a2ff328ff4e1a5d8144da0e4f281f0ca09da8397 100644 (file)
@@ -1458,13 +1458,19 @@ static struct aco_type global = {
 static struct aco_type global_defaults = {
        .type = ACO_GLOBAL,
        .item_offset = offsetof(struct test_config, global_defaults),
-       .category_match = ACO_WHITELIST,
-       .category = "^global_defaults$",
+       .category_match = ACO_WHITELIST_EXACT,
+       .category = "global_defaults",
+};
+static const char *item_blacklist[] = {
+       "global",
+       "global_defaults",
+       NULL,
 };
+
 static struct aco_type item = {
        .type = ACO_ITEM,
-       .category_match = ACO_BLACKLIST,
-       .category = "^(global|global_defaults)$",
+       .category_match = ACO_BLACKLIST_ARRAY,
+       .category = (const char *)item_blacklist,
        .item_alloc = test_item_alloc,
        .item_find = test_item_find,
        .item_offset = offsetof(struct test_config, items),