diff options
| author | Francesco Castellano <francesco.castellano@messagenet.it> | 2015-11-20 15:02:48 +0100 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2016-03-22 10:29:31 -0500 |
| commit | ac669999710231a2a802ee1d290921a870a6fbce (patch) | |
| tree | cfe2b1c21bb4f4c67c0cc4cf67bfed59ed5318fe /channels | |
| parent | c21cee80cc21fd48f29f23eabb7206417c4a7ef6 (diff) | |
chan_sip.c: 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
Diffstat (limited to 'channels')
| -rw-r--r-- | channels/chan_sip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 97cd6abb0..09ab1a196 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -18859,7 +18859,7 @@ static void check_via(struct sip_pvt *p, const struct sip_request *req) c = strchr(via, ' '); if (c) { *c = '\0'; - c = ast_skip_blanks(c+1); + c = ast_strip(c+1); if (strcasecmp(via, "SIP/2.0/UDP") && strcasecmp(via, "SIP/2.0/TCP") && strcasecmp(via, "SIP/2.0/TLS")) { ast_log(LOG_WARNING, "Don't know how to respond via '%s'\n", via); return; |