Misc minor fixes in reqresp_parser.c and chan_sip.c.
authorRichard Mudgett <rmudgett@digium.com>
Thu, 19 Jan 2012 23:31:17 +0000 (23:31 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Thu, 19 Jan 2012 23:31:17 +0000 (23:31 +0000)
commitae32acfa3efecc59e37219ef6d39d6564354f3a3
tree0bf853b90f1bc9ebdc91ee36447dc46aa45d2110
parentadd6efc20cec066e753879a552f01a5e2a6058fa
Misc minor fixes in reqresp_parser.c and chan_sip.c.

* Fix corner cases in get_calleridname() parsing and ensure that the
output buffer is nul terminated.

* Make get_calleridname() truncate the name it parses if the given buffer
is too small rather than abandoning the parse and not returning anything
for the name.  Adjusted get_calleridname_test() unit test to handle the
truncation change.

* Fix get_in_brackets_test() unit test to check the results of
get_in_brackets() correctly.

* Fix parse_name_andor_addr() to not return the address of a local buffer.
This function is currently not used.

* Fix potential NULL pointer dereference in sip_sendtext().

* No need to memset(calleridname) in check_user_full() or tmp_name in
get_name_and_number() because get_calleridname() ensures that it is nul
terminated.

* Reply with an accurate response if get_msg_text() fails in
receive_message().  This is academic in v1.8 because get_msg_text() can
never fail.
........

Merged revisions 351618 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 351646 from http://svn.asterisk.org/svn/asterisk/branches/10

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@351667 65c4cc65-6c06-0410-ace0-fbb531ad65f3
channels/chan_sip.c
channels/sip/reqresp_parser.c