chan_sip: Update the via header when relaying SMS MESSAGE

Prior to this change, certain conditions for sending the message would
result in an address of '(null)' being used in the via header of the
SIP message because a NULl value of pvt->ourip was used when initially
generating the via header. This is fixed by adding a call to build_via
when the address is set before sending the message.

(closes issue ASTERISK-21148)
Reported by: Zhi Cheng
Patches:
	700-sip_msg_send_via_fix.patch uploaded by Zhi Cheng (license 6475)

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@382739 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 5bbdf4e2fd1e1704ab20b2582ee87ee2e4cc7ff0..c2625fc60a01db571a7f6694dda5b240a5185a8f 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -26757,6 +26757,7 @@ static int sip_msg_send(const struct ast_msg *msg, const char *to, const char *f
                ast_string_field_set(pvt, username, to_user);
        }
        ast_sip_ouraddrfor(&pvt->sa, &pvt->ourip, pvt);
+       build_via(pvt);
        ast_set_flag(&pvt->flags[0], SIP_OUTGOING);
 
        /* XXX Does pvt->expiry need to be set? */