diff options
author | Jenkins2 <jenkins2@gerrit.asterisk.org> | 2017-07-05 15:32:40 -0500 |
---|---|---|
committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2017-07-05 15:32:40 -0500 |
commit | 16f0fa52c05e14c050db014bb7d555edb261fe1d (patch) | |
tree | 679e9dd12fd4b571a1ed02f4a7ff926ee7bfca43 /res | |
parent | d2b32cd00938ed2717565c14395f10cc90875a14 (diff) | |
parent | 65a5ac0168ae39340432d60e4fffabc8688a3ca9 (diff) |
Merge "pjsip_distributor.c: Fix unidentified_requests hash functions."
Diffstat (limited to 'res')
-rw-r--r-- | res/res_pjsip/pjsip_distributor.c | 26 |
1 files changed, 18 insertions, 8 deletions
diff --git a/res/res_pjsip/pjsip_distributor.c b/res/res_pjsip/pjsip_distributor.c index cca26a83c..2c9bcbf6e 100644 --- a/res/res_pjsip/pjsip_distributor.c +++ b/res/res_pjsip/pjsip_distributor.c @@ -837,7 +837,7 @@ static int suspects_compare(void *obj, void *arg, int flags) /* Fall through */ case OBJ_SEARCH_KEY: if (strcmp(object_left->src_name, right_key) == 0) { - cmp = CMP_MATCH | CMP_STOP; + cmp = CMP_MATCH; } break; case OBJ_SEARCH_PARTIAL_KEY: @@ -852,15 +852,25 @@ static int suspects_compare(void *obj, void *arg, int flags) return cmp; } -static int suspects_hash(const void *obj, int flags) { - const struct unidentified_request *object_left = obj; +static int suspects_hash(const void *obj, int flags) +{ + const struct unidentified_request *object; + const char *key; - if (flags & OBJ_SEARCH_OBJECT) { - return ast_str_hash(object_left->src_name); - } else if (flags & OBJ_SEARCH_KEY) { - return ast_str_hash(obj); + switch (flags & OBJ_SEARCH_MASK) { + case OBJ_SEARCH_KEY: + key = obj; + break; + case OBJ_SEARCH_OBJECT: + object = obj; + key = object->src_name; + break; + default: + /* Hash can only work on something with a full key. */ + ast_assert(0); + return 0; } - return -1; + return ast_str_hash(key); } static struct ao2_container *cli_unid_get_container(const char *regex) |