From 21eb11c40ebfd1692baa73ca3968c0165fb03368 Mon Sep 17 00:00:00 2001 From: Russell Bryant Date: Fri, 23 Sep 2005 02:40:38 +0000 Subject: 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 --- enum.c | 288 ++++++++++++++++++++++++++++++++--------------------------------- 1 file changed, 144 insertions(+), 144 deletions(-) diff --git a/enum.c b/enum.c index d2483996d..1c47f18e7 100755 --- a/enum.c +++ b/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)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; - - if(number[0] == 'n'){ - strncpy(naptrinput, number+1, sizeof(naptrinput)); - } else { - strncpy(naptrinput, number, sizeof(naptrinput)); - } + /* 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)); + } 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; - - 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; - } - } + 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 (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[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 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... */ -- cgit v1.2.3