[asterisk-bugs] [Asterisk 0013071]: [patch] OPTIONS response on default port.

Asterisk Bug Tracker noreply at bugs.digium.com
Fri Jan 9 15:53:21 CST 2009


A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13071 
====================================================================== 
Reported By:                baron
Assigned To:                kpfleming
====================================================================== 
Project:                    Asterisk
Issue ID:                   13071
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           1.4.21 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-07-14 10:47 CDT
Last Modified:              2009-01-09 15:53 CST
====================================================================== 
Summary:                    [patch] OPTIONS response on default port.
Description: 
handle_request_options doesn't have check_via() function and because the
response go to default port 5060, but should got to port was defined in
Via: Header.

check_via should be also in the folows functions:
handle_request_info()
handle_request_notify()
handle_request_options()
handle_request_refer()
handle_request_message();

====================================================================== 

---------------------------------------------------------------------- 
 (0097379) svnbot (reporter) - 2009-01-09 15:53
 http://bugs.digium.com/view.php?id=13071#c97379 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 168193

_U  trunk/
U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r168193 | mmichelson | 2009-01-09 15:53:19 -0600 (Fri, 09 Jan 2009) | 21
lines

Merged revisions 168128 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r168128 | mmichelson | 2009-01-09 14:08:04 -0600 (Fri, 09 Jan 2009) | 13
lines

Add check_via calls to more request handlers

INFO, NOTIFY, OPTIONS, REFER, and MESSAGE requests
were not checking the topmost Via to determine where
to send the response. Adding check_via calls to those
request handlers solves this.

(closes issue http://bugs.digium.com/view.php?id=13071)
Reported by: baron
Patches:
      check_via.patch uploaded by baron (license 531)
Tested by: baron

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=168193 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-09 15:53 svnbot         Checkin                                      
2009-01-09 15:53 svnbot         Note Added: 0097379                          
======================================================================




More information about the asterisk-bugs mailing list