[Asterisk-code-review] channels/chan sip: Space after port causes unnecessary resol... (asterisk[master])

Francesco Castellano asteriskteam at digium.com
Tue Mar 22 05:17:19 CDT 2016


Francesco Castellano has uploaded a new patch set (#2).

Change subject: channels/chan_sip: Space after port causes unnecessary resolution attempt
......................................................................

channels/chan_sip: Space after port causes unnecessary resolution attempt

check_via() already skips leading blanks where the sent-by address (with the
optional port) should be placed.

Since RFC 3261 allows for blanks between the port ant the Via parameters:
> https://tools.ietf.org/html/rfc3261#section-20.42
(actually it allows a lot of blanks more ;-)). I just switched from
ast_skip_blanks() to ast_strip() on the local copy of the string.

ASTERISK-21301 #close
Change-Id: Ie5b8fe5a07067b7c0dc9bcdd1707e99b23b02b06
---
M channels/chan_sip.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/46/2446/2
-- 
To view, visit https://gerrit.asterisk.org/2446
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie5b8fe5a07067b7c0dc9bcdd1707e99b23b02b06
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Francesco Castellano <francesco.castellano at messagenet.it>



More information about the asterisk-code-review mailing list