diff options
| -rw-r--r-- | channels/sip/include/reqresp_parser.h | 5 | ||||
| -rw-r--r-- | channels/sip/reqresp_parser.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/channels/sip/include/reqresp_parser.h b/channels/sip/include/reqresp_parser.h index 7f9c8f6d9..2543329dd 100644 --- a/channels/sip/include/reqresp_parser.h +++ b/channels/sip/include/reqresp_parser.h @@ -176,6 +176,11 @@ void sip_request_parser_unregister_tests(void); * \param option list * \param unsupported out buffer (optional) * \param unsupported out buffer length (optional) + * + * \note Because this function can be called multiple times, it will append + * whatever options are specified in \c options to \c unsupported. Callers + * of this function should make sure the unsupported buffer is clear before + * calling this function. */ unsigned int parse_sip_options(const char *options, char *unsupported, size_t unsupported_len); diff --git a/channels/sip/reqresp_parser.c b/channels/sip/reqresp_parser.c index aa568a0e9..e5c1ff2bc 100644 --- a/channels/sip/reqresp_parser.c +++ b/channels/sip/reqresp_parser.c @@ -1698,7 +1698,6 @@ unsigned int parse_sip_options(const char *options, char *unsupported, size_t un temp = ast_strdupa(options); ast_debug(3, "Begin: parsing SIP \"Supported: %s\"\n", options); - for (next = temp; next; next = sep) { found = FALSE; supported = FALSE; @@ -1858,6 +1857,7 @@ AST_TEST_DEFINE(sip_parse_options_test) /* Test with unsupported char buffer */ AST_LIST_TRAVERSE(&testdatalist, testdataptr, list) { + memset(unsupported, 0, sizeof(unsupported)); option_profile = parse_sip_options(testdataptr->input_options, unsupported, ARRAY_LEN(unsupported)); if (option_profile != testdataptr->expected_profile || strcmp(unsupported, testdataptr->expected_unsupported)) { |