[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:05 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0098478) svnbot (reporter) - 2009-01-22 14:05
 http://bugs.digium.com/view.php?id=14234#c98478 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 170242

_U  branches/1.6.1/
U   branches/1.6.1/main/rtp.c

------------------------------------------------------------------------
r170242 | file | 2009-01-22 14:05:44 -0600 (Thu, 22 Jan 2009) | 21 lines

Merged revisions 170240 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r170240 | file | 2009-01-22 16:04:39 -0400 (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=170242 

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