[asterisk-bugs] [Asterisk 0014452]: in "_sip_tcp_helper_thread" Buffer is filled with dirty bytes

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://bugs.digium.com/view.php?id=14452 
====================================================================== 
Reported By:                umberto71
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14452
Category:                   Channels/chan_sip/TLS
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0.1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-02-10 10:36 CST
Last Modified:              2009-02-10 10:36 CST
====================================================================== 
Summary:                    in "_sip_tcp_helper_thread" Buffer is filled with
dirty bytes
Description: 
I've seen with aastra phones 51i and 57i that buffer "buf" is not empty
with TLS. 
I suggest to do this patch
.....
	for (;;) {
		memset(req.data, 0, sizeof(req.data));
		memset(buf, 0, sizeof(buf));  //<<<<<<<<<<<<<<<<<<< Patch
		req.len = 0;
		req.ignore = 0;
		req.debug = 0;
.....
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-02-10 10:36 umberto71      New Issue                                    
2009-02-10 10:36 umberto71      Asterisk Version          => 1.6.0.1         
2009-02-10 10:36 umberto71      Regression                => No              
2009-02-10 10:36 umberto71      SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================