fix some gcc4 pointer signedness warnings and clean up some formatting
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@6628 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
parent
8087701923
commit
21eb11c40e
284
enum.c
284
enum.c
|
@ -101,11 +101,11 @@ static int parse_ie(char *data, int maxdatalen, char *src, int srclen)
|
|||
static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *answer, int len, char *naptrinput)
|
||||
{
|
||||
|
||||
char tech_return[80];
|
||||
char tech_return[80];
|
||||
char *oanswer = answer;
|
||||
char flags[512] = "";
|
||||
char services[512] = "";
|
||||
unsigned char *p;
|
||||
char *p;
|
||||
char regexp[512] = "";
|
||||
char repl[512] = "";
|
||||
char temp[512] = "";
|
||||
|
@ -118,7 +118,7 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
regex_t preg;
|
||||
regmatch_t pmatch[9];
|
||||
|
||||
tech_return[0] = '\0';
|
||||
tech_return[0] = '\0';
|
||||
|
||||
dst[0] = '\0';
|
||||
|
||||
|
@ -130,30 +130,30 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
len -= sizeof(struct naptr);
|
||||
if ((res = parse_ie(flags, sizeof(flags) - 1, answer, len)) < 0) {
|
||||
ast_log(LOG_WARNING, "Failed to get flags from NAPTR record\n");
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
}
|
||||
if ((res = parse_ie(services, sizeof(services) - 1, answer, len)) < 0) {
|
||||
ast_log(LOG_WARNING, "Failed to get services from NAPTR record\n");
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
}
|
||||
if ((res = parse_ie(regexp, sizeof(regexp) - 1, answer, len)) < 0) {
|
||||
ast_log(LOG_WARNING, "Failed to get regexp from NAPTR record\n");
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
return -1;
|
||||
} else {
|
||||
answer += res;
|
||||
len -= res;
|
||||
}
|
||||
|
||||
if ((res = dn_expand((unsigned char *)oanswer, (unsigned char *)answer + len, (unsigned char *)answer, repl, sizeof(repl) - 1)) < 0) {
|
||||
ast_log(LOG_WARNING, "Failed to expand hostname\n");
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
if (option_debug > 2) /* Advanced NAPTR debugging */
|
||||
ast_log(LOG_DEBUG, "NAPTR input='%s', flags='%s', services='%s', regexp='%s', repl='%s'\n",
|
||||
|
@ -164,27 +164,27 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
return -1;
|
||||
}
|
||||
|
||||
p = strstr(services, "e2u+");
|
||||
if(p == NULL)
|
||||
p = strstr(services, "E2U+");
|
||||
if(p){
|
||||
p = p + 4;
|
||||
if(strchr(p, ':')){
|
||||
p = strchr(p, ':') + 1;
|
||||
}
|
||||
ast_copy_string(tech_return, p, sizeof(tech_return));
|
||||
p = strstr(services, "e2u+");
|
||||
if (p == NULL)
|
||||
p = strstr(services, "E2U+");
|
||||
if (p){
|
||||
p = p + 4;
|
||||
if (strchr(p, ':')){
|
||||
p = strchr(p, ':') + 1;
|
||||
}
|
||||
ast_copy_string(tech_return, p, sizeof(tech_return));
|
||||
} else {
|
||||
|
||||
p = strstr(services, "+e2u");
|
||||
if(p == NULL)
|
||||
p = strstr(services, "+E2U");
|
||||
if(p){
|
||||
*p = 0;
|
||||
p = strchr(services, ':');
|
||||
if(p)
|
||||
*p = 0;
|
||||
ast_copy_string(tech_return, services, sizeof(tech_return));
|
||||
}
|
||||
p = strstr(services, "+e2u");
|
||||
if (p == NULL)
|
||||
p = strstr(services, "+E2U");
|
||||
if (p) {
|
||||
*p = 0;
|
||||
p = strchr(services, ':');
|
||||
if (p)
|
||||
*p = 0;
|
||||
ast_copy_string(tech_return, services, sizeof(tech_return));
|
||||
}
|
||||
}
|
||||
|
||||
/* DEDBUGGING STUB
|
||||
|
@ -195,7 +195,7 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
if (regexp_len < 7) {
|
||||
ast_log(LOG_WARNING, "Regex too short to be meaningful.\n");
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
delim = regexp[0];
|
||||
|
@ -238,9 +238,9 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
}
|
||||
regfree(&preg);
|
||||
|
||||
d = temp;
|
||||
d_len--;
|
||||
while( *subst && (d_len > 0) ) {
|
||||
d = temp;
|
||||
d_len--;
|
||||
while (*subst && (d_len > 0)) {
|
||||
if ((subst[0] == '\\') && isdigit(subst[1]) && (pmatch[subst[1]-'0'].rm_so != -1)) {
|
||||
backref = subst[1]-'0';
|
||||
size = pmatch[backref].rm_eo - pmatch[backref].rm_so;
|
||||
|
@ -264,32 +264,32 @@ static int parse_naptr(char *dst, int dstsize, char *tech, int techsize, char *a
|
|||
ast_copy_string(dst, temp, dstsize);
|
||||
dst[dstsize - 1] = '\0';
|
||||
|
||||
if(*tech != '\0'){ /* check if it is requested NAPTR */
|
||||
if(!strncasecmp(tech, "ALL", techsize)){
|
||||
return 1; /* return or count any RR */
|
||||
}
|
||||
if(!strncasecmp(tech_return, tech, sizeof(tech_return)<techsize?sizeof(tech_return):techsize)){
|
||||
ast_copy_string(tech, tech_return, techsize);
|
||||
return 1; /* we got out RR */
|
||||
} else { /* go to the next RR in the DNS answer */
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
if (*tech != '\0'){ /* check if it is requested NAPTR */
|
||||
if (!strncasecmp(tech, "ALL", techsize)){
|
||||
return 1; /* return or count any RR */
|
||||
}
|
||||
if (!strncasecmp(tech_return, tech, sizeof(tech_return)<techsize?sizeof(tech_return):techsize)){
|
||||
ast_copy_string(tech, tech_return, techsize);
|
||||
return 1; /* we got out RR */
|
||||
} else { /* go to the next RR in the DNS answer */
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
|
||||
/* tech was not specified, return first parsed RR */
|
||||
ast_copy_string(tech, tech_return, techsize);
|
||||
/* tech was not specified, return first parsed RR */
|
||||
ast_copy_string(tech, tech_return, techsize);
|
||||
|
||||
return 1;
|
||||
return 1;
|
||||
}
|
||||
|
||||
/* do not return requested value, just count RRs and return thei number in dst */
|
||||
#define ENUMLOOKUP_OPTIONS_COUNT 1
|
||||
|
||||
struct enum_naptr_rr {
|
||||
struct naptr naptr; /* order and preference of RR */
|
||||
char *result; /* result of naptr parsing,e.g.: tel:+5553 */
|
||||
char *tech; /* Technology (from URL scheme) */
|
||||
int sort_pos; /* sort position */
|
||||
struct naptr naptr; /* order and preference of RR */
|
||||
char *result; /* result of naptr parsing,e.g.: tel:+5553 */
|
||||
char *tech; /* Technology (from URL scheme) */
|
||||
int sort_pos; /* sort position */
|
||||
};
|
||||
|
||||
struct enum_context {
|
||||
|
@ -300,10 +300,10 @@ struct enum_context {
|
|||
char *txt; /* TXT record in TXT lookup */
|
||||
int txtlen; /* Length */
|
||||
char *naptrinput; /* The number to lookup */
|
||||
int position; /* used as counter for RRs or specifies position of required RR */
|
||||
int options; /* options , see ENUMLOOKUP_OPTIONS_* defined above */
|
||||
struct enum_naptr_rr *naptr_rrs; /* array of parsed NAPTR RRs */
|
||||
int naptr_rrs_count; /* Size of array naptr_rrs */
|
||||
int position; /* used as counter for RRs or specifies position of required RR */
|
||||
int options; /* options , see ENUMLOOKUP_OPTIONS_* defined above */
|
||||
struct enum_naptr_rr *naptr_rrs; /* array of parsed NAPTR RRs */
|
||||
int naptr_rrs_count; /* Size of array naptr_rrs */
|
||||
};
|
||||
|
||||
/*--- txt_callback: Callback for TXT record lookup */
|
||||
|
@ -350,16 +350,16 @@ static int enum_callback(void *context, char *answer, int len, char *fullanswer)
|
|||
|
||||
res = parse_naptr(c->dst, c->dstlen, c->tech, c->techlen, answer, len, c->naptrinput);
|
||||
|
||||
if(res < 0){
|
||||
if (res < 0) {
|
||||
ast_log(LOG_WARNING, "Failed to parse naptr :(\n");
|
||||
return -1;
|
||||
} else if(res > 0 && !ast_strlen_zero(c->dst)){ /* ok, we got needed NAPTR */
|
||||
if(c->options & ENUMLOOKUP_OPTIONS_COUNT){ /* counting RRs */
|
||||
} else if (res > 0 && !ast_strlen_zero(c->dst)){ /* ok, we got needed NAPTR */
|
||||
if (c->options & ENUMLOOKUP_OPTIONS_COUNT){ /* counting RRs */
|
||||
c->position++;
|
||||
snprintf(c->dst, c->dstlen, "%d", c->position);
|
||||
} else {
|
||||
p = realloc(c->naptr_rrs, sizeof(struct enum_naptr_rr)*(c->naptr_rrs_count+1));
|
||||
if(p){
|
||||
if (p) {
|
||||
c->naptr_rrs = (struct enum_naptr_rr*)p;
|
||||
memcpy(&c->naptr_rrs[c->naptr_rrs_count].naptr, answer, sizeof(struct naptr));
|
||||
/* printf("order=%d, pref=%d\n", ntohs(c->naptr_rrs[c->naptr_rrs_count].naptr.order), ntohs(c->naptr_rrs[c->naptr_rrs_count].naptr.pref)); */
|
||||
|
@ -373,7 +373,7 @@ static int enum_callback(void *context, char *answer, int len, char *fullanswer)
|
|||
return 0;
|
||||
}
|
||||
|
||||
if(c->options & ENUMLOOKUP_OPTIONS_COUNT){ /* counting RRs */
|
||||
if (c->options & ENUMLOOKUP_OPTIONS_COUNT) { /* counting RRs */
|
||||
snprintf(c->dst, c->dstlen, "%d", c->position);
|
||||
}
|
||||
|
||||
|
@ -385,86 +385,86 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
{
|
||||
struct enum_context context;
|
||||
char tmp[259 + 512];
|
||||
char naptrinput[512];
|
||||
char naptrinput[512];
|
||||
int pos = strlen(number) - 1;
|
||||
int newpos = 0;
|
||||
int ret = -1;
|
||||
struct enum_search *s = NULL;
|
||||
int version = -1;
|
||||
/* for ISN rewrite */
|
||||
char *p1 = NULL;
|
||||
char *p2 = NULL;
|
||||
int k = 0;
|
||||
int i = 0;
|
||||
int z = 0;
|
||||
/* for ISN rewrite */
|
||||
char *p1 = NULL;
|
||||
char *p2 = NULL;
|
||||
int k = 0;
|
||||
int i = 0;
|
||||
int z = 0;
|
||||
|
||||
if(number[0] == 'n'){
|
||||
strncpy(naptrinput, number+1, sizeof(naptrinput));
|
||||
} else {
|
||||
strncpy(naptrinput, number, sizeof(naptrinput));
|
||||
}
|
||||
if (number[0] == 'n') {
|
||||
strncpy(naptrinput, number+1, sizeof(naptrinput));
|
||||
} else {
|
||||
strncpy(naptrinput, number, sizeof(naptrinput));
|
||||
}
|
||||
|
||||
context.naptrinput = naptrinput; /* The number */
|
||||
context.dst = dst; /* Return string */
|
||||
context.dstlen = dstlen;
|
||||
context.tech = tech;
|
||||
context.tech = tech;
|
||||
context.techlen = techlen;
|
||||
context.options = 0;
|
||||
context.position = 1;
|
||||
context.naptr_rrs = NULL;
|
||||
context.naptr_rrs_count = 0;
|
||||
context.options = 0;
|
||||
context.position = 1;
|
||||
context.naptr_rrs = NULL;
|
||||
context.naptr_rrs_count = 0;
|
||||
|
||||
if(options != NULL){
|
||||
if(*options == 'c'){
|
||||
context.options = ENUMLOOKUP_OPTIONS_COUNT;
|
||||
context.position = 0;
|
||||
} else {
|
||||
context.position = atoi(options);
|
||||
if(context.position < 1)
|
||||
context.position = 1;
|
||||
}
|
||||
}
|
||||
if (options != NULL){
|
||||
if (*options == 'c'){
|
||||
context.options = ENUMLOOKUP_OPTIONS_COUNT;
|
||||
context.position = 0;
|
||||
} else {
|
||||
context.position = atoi(options);
|
||||
if (context.position < 1)
|
||||
context.position = 1;
|
||||
}
|
||||
}
|
||||
|
||||
if (pos > 128)
|
||||
pos = 128;
|
||||
|
||||
/* ISN rewrite */
|
||||
p1 = strchr(number, '*');
|
||||
/* ISN rewrite */
|
||||
p1 = strchr(number, '*');
|
||||
|
||||
if(number[0] == 'n'){ /* do not perform ISN rewrite ('n' is testing flag) */
|
||||
p1 = NULL;
|
||||
k = 1; /* strip 'n' from number */
|
||||
}
|
||||
if (number[0] == 'n') { /* do not perform ISN rewrite ('n' is testing flag) */
|
||||
p1 = NULL;
|
||||
k = 1; /* strip 'n' from number */
|
||||
}
|
||||
|
||||
if(p1 != NULL){
|
||||
p2 = p1+1;
|
||||
while(p1 > number){
|
||||
p1--;
|
||||
tmp[newpos++] = *p1;
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
if(*p2){
|
||||
while(*p2 && newpos < 128){
|
||||
tmp[newpos++] = *p2;
|
||||
p2++;
|
||||
}
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
if (p1 != NULL) {
|
||||
p2 = p1+1;
|
||||
while (p1 > number){
|
||||
p1--;
|
||||
tmp[newpos++] = *p1;
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
if (*p2) {
|
||||
while(*p2 && newpos < 128){
|
||||
tmp[newpos++] = *p2;
|
||||
p2++;
|
||||
}
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
|
||||
} else {
|
||||
while(pos >= k) {
|
||||
if(isdigit(number[pos])){
|
||||
tmp[newpos++] = number[pos];
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
pos--;
|
||||
}
|
||||
} else {
|
||||
while (pos >= k) {
|
||||
if (isdigit(number[pos])) {
|
||||
tmp[newpos++] = number[pos];
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
pos--;
|
||||
}
|
||||
}
|
||||
|
||||
if (chan && ast_autoservice_start(chan) < 0)
|
||||
return -1;
|
||||
|
||||
for(;;) {
|
||||
for (;;) {
|
||||
ast_mutex_lock(&enumlock);
|
||||
if (version != enumver) {
|
||||
/* Ooh, a reload... */
|
||||
|
@ -473,9 +473,9 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
} else {
|
||||
s = s->next;
|
||||
}
|
||||
if(suffix != NULL){
|
||||
strncpy(tmp + newpos, suffix, sizeof(tmp) - newpos - 1);
|
||||
} else if (s) {
|
||||
if (suffix != NULL) {
|
||||
strncpy(tmp + newpos, suffix, sizeof(tmp) - newpos - 1);
|
||||
} else if (s) {
|
||||
strncpy(tmp + newpos, s->toplev, sizeof(tmp) - newpos - 1);
|
||||
}
|
||||
ast_mutex_unlock(&enumlock);
|
||||
|
@ -484,7 +484,7 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
ret = ast_search_dns(&context, tmp, C_IN, T_NAPTR, enum_callback);
|
||||
if (ret > 0)
|
||||
break;
|
||||
if(suffix != NULL)
|
||||
if (suffix != NULL)
|
||||
break;
|
||||
}
|
||||
if (ret < 0) {
|
||||
|
@ -492,12 +492,12 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
ret = 0;
|
||||
}
|
||||
|
||||
if(context.naptr_rrs_count >= context.position && ! (context.options & ENUMLOOKUP_OPTIONS_COUNT)){
|
||||
if (context.naptr_rrs_count >= context.position && ! (context.options & ENUMLOOKUP_OPTIONS_COUNT)) {
|
||||
/* sort array by NAPTR order/preference */
|
||||
for(k=0; k<context.naptr_rrs_count; k++){
|
||||
for(i=0; i<context.naptr_rrs_count; i++){
|
||||
for (k=0; k<context.naptr_rrs_count; k++) {
|
||||
for (i=0; i<context.naptr_rrs_count; i++) {
|
||||
/* use order first and then preference to compare */
|
||||
if((ntohs(context.naptr_rrs[k].naptr.order) < ntohs(context.naptr_rrs[i].naptr.order)
|
||||
if ((ntohs(context.naptr_rrs[k].naptr.order) < ntohs(context.naptr_rrs[i].naptr.order)
|
||||
&& context.naptr_rrs[k].sort_pos > context.naptr_rrs[i].sort_pos)
|
||||
|| (ntohs(context.naptr_rrs[k].naptr.order) > ntohs(context.naptr_rrs[i].naptr.order)
|
||||
&& context.naptr_rrs[k].sort_pos < context.naptr_rrs[i].sort_pos)){
|
||||
|
@ -506,8 +506,8 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
context.naptr_rrs[i].sort_pos = z;
|
||||
continue;
|
||||
}
|
||||
if(ntohs(context.naptr_rrs[k].naptr.order) == ntohs(context.naptr_rrs[i].naptr.order)){
|
||||
if((ntohs(context.naptr_rrs[k].naptr.pref) < ntohs(context.naptr_rrs[i].naptr.pref)
|
||||
if (ntohs(context.naptr_rrs[k].naptr.order) == ntohs(context.naptr_rrs[i].naptr.order)) {
|
||||
if ((ntohs(context.naptr_rrs[k].naptr.pref) < ntohs(context.naptr_rrs[i].naptr.pref)
|
||||
&& context.naptr_rrs[k].sort_pos > context.naptr_rrs[i].sort_pos)
|
||||
|| (ntohs(context.naptr_rrs[k].naptr.pref) > ntohs(context.naptr_rrs[i].naptr.pref)
|
||||
&& context.naptr_rrs[k].sort_pos < context.naptr_rrs[i].sort_pos)){
|
||||
|
@ -518,26 +518,26 @@ int ast_get_enum(struct ast_channel *chan, const char *number, char *dst, int ds
|
|||
}
|
||||
}
|
||||
}
|
||||
for(k=0; k<context.naptr_rrs_count; k++){
|
||||
if(context.naptr_rrs[k].sort_pos == context.position-1){
|
||||
for (k=0; k<context.naptr_rrs_count; k++) {
|
||||
if (context.naptr_rrs[k].sort_pos == context.position-1) {
|
||||
ast_copy_string(context.dst, context.naptr_rrs[k].result, dstlen);
|
||||
ast_copy_string(context.tech, context.naptr_rrs[k].tech, techlen);
|
||||
break;
|
||||
}
|
||||
}
|
||||
} else if( !(context.options & ENUMLOOKUP_OPTIONS_COUNT) ) {
|
||||
} else if (!(context.options & ENUMLOOKUP_OPTIONS_COUNT)) {
|
||||
context.dst[0] = 0;
|
||||
}
|
||||
|
||||
if (chan)
|
||||
ret |= ast_autoservice_stop(chan);
|
||||
|
||||
for(k=0; k<context.naptr_rrs_count; k++){
|
||||
free(context.naptr_rrs[k].result);
|
||||
free(context.naptr_rrs[k].tech);
|
||||
}
|
||||
for (k=0; k<context.naptr_rrs_count; k++) {
|
||||
free(context.naptr_rrs[k].result);
|
||||
free(context.naptr_rrs[k].tech);
|
||||
}
|
||||
|
||||
free(context.naptr_rrs);
|
||||
free(context.naptr_rrs);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
@ -568,7 +568,7 @@ int ast_get_txt(struct ast_channel *chan, const char *number, char *dst, int dst
|
|||
|
||||
if (pos > 128)
|
||||
pos = 128;
|
||||
while(pos >= 0) {
|
||||
while (pos >= 0) {
|
||||
tmp[newpos++] = number[pos--];
|
||||
tmp[newpos++] = '.';
|
||||
}
|
||||
|
@ -576,7 +576,7 @@ int ast_get_txt(struct ast_channel *chan, const char *number, char *dst, int dst
|
|||
if (chan && ast_autoservice_start(chan) < 0)
|
||||
return -1;
|
||||
|
||||
for(;;) {
|
||||
for (;;) {
|
||||
ast_mutex_lock(&enumlock);
|
||||
if (version != enumver) {
|
||||
/* Ooh, a reload... */
|
||||
|
|
Loading…
Reference in New Issue