[asterisk-bugs] [Asterisk 0014929]: [patch] blank lines in _sip_tcp_helper_thread caused by sip dummy packages

[Asterisk Bug Tracker]

The following issue has been UPDATED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14929 
====================================================================== 
Reported By:                vrban
Assigned To:                russell
====================================================================== 
Project:                    Asterisk
Issue ID:                   14929
Category:                   Channels/chan_sip/TCP-TLS
Reproducibility:            have not tried
Severity:                   tweak
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 189204 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-04-18 03:46 CDT
Last Modified:              2009-09-10 16:39 CDT
====================================================================== 
Summary:                    [patch] blank lines in _sip_tcp_helper_thread caused
by sip dummy packages
Description: 
hello, i tested TLS/TCP with a softphone called PhonerLite (based on oSIP)
And when i used it, the TCP buffer in _sip_tcp_helper_thread always was
filled with a blank line. Because the softphone broadcast empty SIP dummy
packages as NAT opener.

This was very annoying, because the dummy SIP package contents only a
single "\r\n", with the effect, that a real SIP messages was getting this
extra blank line from the dummy package because it was still in the buffer,
and so the parse_request and get_header function was not capable to work
proper. I my case this was the reason the return value for the
Content-Length was always 0 and
no sdp was read.

attached is a patch for this issue, but i am not sure if this should go
into chan_sip because it is a fix for crappy SIP dummy packages send by
other. But it is good to know what could go wrong in the
_sip_tcp_helper_thread function and have a possible solution.
====================================================================== 

---------------------------------------------------------------------- 
 (0110537) russell (administrator) - 2009-09-10 16:39
 https://issues.asterisk.org/view.php?id=14929#c110537 
---------------------------------------------------------------------- 
I believe this should be fixed by revision 203779 in Asterisk trunk. 
Please reopen if you still have a problem.

Thanks! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-09-10 16:39 russell        Note Added: 0110537                          
2009-09-10 16:39 russell        Status                   new => resolved     
2009-09-10 16:39 russell        Resolution               open => fixed       
2009-09-10 16:39 russell        Assigned To               => russell         
2009-09-10 16:39 russell        Status                   resolved => closed  
======================================================================