[asterisk-bugs] [Asterisk 0014234]: [patch] Log and debug messages in ast_rtp_destroy can cause a crash

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14234 
====================================================================== 
Reported By:                jcovert
Assigned To:                file
====================================================================== 
Project:                    Asterisk
Issue ID:                   14234
Category:                   Core/RTP
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.0.3 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-01-13 17:43 CST
Last Modified:              2009-01-22 14:04 CST
====================================================================== 
Summary:                    [patch] Log and debug messages in ast_rtp_destroy
can cause a crash
Description: 
It's possible for rtp->rtcp to be zero, in which case a reference to
rtp->rtcp->whatever is not a good idea.

When app_asr ("just digits") calls ast_rtp_destroy, a crash happens every
time.

Patch included fixes this.
====================================================================== 

---------------------------------------------------------------------- 
 (0098476) svnbot (reporter) - 2009-01-22 14:04
 http://bugs.digium.com/view.php?id=14234#c98476 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 170240

_U  trunk/
U   trunk/main/rtp.c

------------------------------------------------------------------------
r170240 | file | 2009-01-22 14:04:16 -0600 (Thu, 22 Jan 2009) | 14 lines

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

........
  r170239 | file | 2009-01-22 16:02:35 -0400 (Thu, 22 Jan 2009) | 7 lines
  
  Don't crash if RTCP is not enabled on an RTP structure but statistics
are output.
  (closes issue http://bugs.digium.com/view.php?id=14234)
  Reported by: jcovert
  Patches:
        rtp.c.patch-1.6.0.3 uploaded by jcovert (license 551)
        rtp.c.patch-svn-165599 uploaded by jcovert (license 551)
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-22 14:04 svnbot         Checkin                                      
2009-01-22 14:04 svnbot         Note Added: 0098476                          
======================================================================