From 96c231fc1831d9927fe4773e553a7bce9b18a40a Mon Sep 17 00:00:00 2001 From: Jonathan Rose <jrose@digium.com> Date: Fri, 8 Mar 2013 20:16:43 +0000 Subject: [PATCH] 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 --- channels/chan_sip.c | 1 + 1 file changed, 1 insertion(+) diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 5bbdf4e2fd..c2625fc60a 100644 --- 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? */ -- GitLab