res_sorcery_astdb.c: Fix get multiple records by regex.
authorRichard Mudgett <rmudgett@digium.com>
Mon, 9 Dec 2013 18:50:20 +0000 (18:50 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Mon, 9 Dec 2013 18:50:20 +0000 (18:50 +0000)
* Fix sorcery_astdb_retrieve_regex() pattern matching.  Let the regexec()
function match the stored key values instead of having astdb prefilter
them.  Previoiusly you could only use a simple regex pattern when the
pattern began with '^'.

* Fix off nominal memory leak in sorcery_astdb_retrieve_regex().
........

Merged revisions 403545 from http://svn.asterisk.org/svn/asterisk/branches/12

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@403546 65c4cc65-6c06-0410-ace0-fbb531ad65f3

res/res_sorcery_astdb.c

index ecea885..61a607f 100644 (file)
@@ -218,20 +218,14 @@ static void sorcery_astdb_retrieve_multiple(const struct ast_sorcery *sorcery, v
 static void sorcery_astdb_retrieve_regex(const struct ast_sorcery *sorcery, void *data, const char *type, struct ao2_container *objects, const char *regex)
 {
        const char *prefix = data;
-       char family[strlen(prefix) + strlen(type) + 2], tree[strlen(regex) + 1];
+       char family[strlen(prefix) + strlen(type) + 2];
        RAII_VAR(struct ast_db_entry *, entries, NULL, ast_db_freetree);
        regex_t expression;
        struct ast_db_entry *entry;
 
        snprintf(family, sizeof(family), "%s/%s", prefix, type);
 
-       if (regex[0] == '^') {
-               snprintf(tree, sizeof(tree), "%s%%", regex + 1);
-       } else {
-               tree[0] = '\0';
-       }
-
-       if (!(entries = ast_db_gettree(family, tree)) || regcomp(&expression, regex, REG_EXTENDED | REG_NOSUB)) {
+       if (!(entries = ast_db_gettree(family, NULL)) || regcomp(&expression, regex, REG_EXTENDED | REG_NOSUB)) {
                return;
        }
 
@@ -249,6 +243,7 @@ static void sorcery_astdb_retrieve_regex(const struct ast_sorcery *sorcery, void
                        !(objset = sorcery_json_to_objectset(json)) ||
                        !(object = ast_sorcery_alloc(sorcery, type, key)) ||
                        ast_sorcery_objectset_apply(sorcery, object, objset)) {
+                       regfree(&expression);
                        return;
                }