diff --git a/channels/chan_iax2.c b/channels/chan_iax2.c index 1251c3c73b769e6b840982d08fcc02ca61f4ef32..92c7e8c8e0b6fab618197029bf9744c471469485 100644 --- a/channels/chan_iax2.c +++ b/channels/chan_iax2.c @@ -1975,13 +1975,14 @@ static char *complete_iax2_show_peer(char *line, char *word, int pos, int state) int which = 0; struct iax2_peer *p; char *res = NULL; + int wordlen = strlen(word); /* 0 - iax2; 1 - show; 2 - peer; 3 - <peername> */ - if(pos == 3) { + if (pos == 3) { ast_mutex_lock(&peerl.lock); - for(p = peerl.peers ; p ; p = p->next) { - if(!strncasecmp(p->name, word, strlen(word))) { - if(++which > state) { + for (p = peerl.peers ; p ; p = p->next) { + if (!strncasecmp(p->name, word, wordlen)) { + if (++which > state) { res = strdup(p->name); break; } diff --git a/channels/iax2-provision.c b/channels/iax2-provision.c index c1b6f4b3d20ba3b1fab35144e4a78d14f794b430..420e7921392d09a958c9ad07044d43b4b72efe1f 100644 --- a/channels/iax2-provision.c +++ b/channels/iax2-provision.c @@ -156,21 +156,20 @@ char *iax_prov_complete_template(char *line, char *word, int pos, int state) { struct iax_template *c; int which=0; - char *ret; + char *ret = NULL; + int wordlen = strlen(word); + ast_mutex_lock(&provlock); - c = templates; - while(c) { - if (!strncasecmp(word, c->name, strlen(word))) { - if (++which > state) + for (c = templates; c; c = c->next) { + if (!strncasecmp(word, c->name, wordlen)) { + if (++which > state) { + ret = strdup(c->name); break; + } } - c = c->next; } - if (c) { - ret = strdup(c->name); - } else - ret = NULL; ast_mutex_unlock(&provlock); + return ret; }