diff --git a/funcs/func_base64.c b/funcs/func_base64.c index a68cf9bdab6126998794cec062e6894f94f41664..34c6ec6ae90c74ec4fedab64174b5a83de948d94 100644 --- a/funcs/func_base64.c +++ b/funcs/func_base64.c @@ -79,13 +79,26 @@ static int base64_helper(struct ast_channel *chan, const char *cmd, char *data, ast_str_update(*str); } } else { + int decoded_len; if (buf) { - ast_base64decode((unsigned char *) buf, data, len); + decoded_len = ast_base64decode((unsigned char *) buf, data, len); + /* add a terminating null at the end of buf, or at the + * end of our decoded string, which ever is less */ + buf[decoded_len <= (len - 1) ? decoded_len : len - 1] = '\0'; } else { if (len >= 0) { ast_str_make_space(str, len ? len : ast_str_strlen(*str) + strlen(data) * 3 / 4 + 2); } - ast_base64decode((unsigned char *) ast_str_buffer(*str) + ast_str_strlen(*str), data, ast_str_size(*str) - ast_str_strlen(*str)); + decoded_len = ast_base64decode((unsigned char *) ast_str_buffer(*str) + ast_str_strlen(*str), data, ast_str_size(*str) - ast_str_strlen(*str)); + if (len) + /* add a terminating null at the end of our + * buffer, or at the end of our decoded string, + * which ever is less */ + ast_str_buffer(*str)[decoded_len <= (len - 1) ? decoded_len : len - 1] = '\0'; + else + /* space for the null is allocated above */ + ast_str_buffer(*str)[decoded_len] = '\0'; + ast_str_update(*str); } } diff --git a/main/utils.c b/main/utils.c index b50fabc0b423d996bc174079cde8a69ce97f421a..e0005a6b73ce75691dbba10ec2794b316d5ca1d4 100644 --- a/main/utils.c +++ b/main/utils.c @@ -267,7 +267,7 @@ int ast_base64decode(unsigned char *dst, const char *src, int max) unsigned int byte = 0; unsigned int bits = 0; int incnt = 0; - while (*src && (cnt < max)) { + while(*src && *src != '=' && (cnt < max)) { /* Shift in 6 bits of input */ byte <<= 6; byte |= (b2a[(int)(*src)]) & 0x3f;