diff options
author | lmadsen <lmadsen@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-01-17 18:58:50 +0000 |
---|---|---|
committer | lmadsen <lmadsen@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-01-17 18:58:50 +0000 |
commit | 810610673ba73bbf2fab48cb174d29d0a889425c (patch) | |
tree | 74012a6992e7f0f30063b440c6d36c5398854cda /main | |
parent | fb0ee12f0b73b0a8a68f70b562ee30fbd15f64b4 (diff) |
AST-2011-001
git-svn-id: http://svn.digium.com/svn/asterisk/tags/1.6.2.16.1@302148 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main')
-rw-r--r-- | main/utils.c | 27 |
1 files changed, 13 insertions, 14 deletions
diff --git a/main/utils.c b/main/utils.c index 10b9a4d8d..f5a218c4d 100644 --- a/main/utils.c +++ b/main/utils.c @@ -385,28 +385,27 @@ char *ast_uri_encode(const char *string, char *outbuf, int buflen, int doreserve char *reserved = ";/?:@&=+$,# "; /* Reserved chars */ const char *ptr = string; /* Start with the string */ - char *out = NULL; - char *buf = NULL; + char *out = outbuf; - ast_copy_string(outbuf, string, buflen); - - /* If there's no characters to convert, just go through and don't do anything */ - while (*ptr) { + /* If there's no characters to convert, just go through and copy the string */ + while (*ptr && out - outbuf < buflen - 1) { if ((*ptr < 32) || (doreserved && strchr(reserved, *ptr))) { - /* Oops, we need to start working here */ - if (!buf) { - buf = outbuf; - out = buf + (ptr - string) ; /* Set output ptr */ + if (out - outbuf >= buflen - 3) { + break; } + out += sprintf(out, "%%%02x", (unsigned char) *ptr); - } else if (buf) { - *out = *ptr; /* Continue copying the string */ + } else { + *out = *ptr; /* copy the character */ out++; - } + } ptr++; } - if (buf) + + if (buflen) { *out = '\0'; + } + return outbuf; } |