[asterisk-bugs] [Asterisk 0014080]: [patch] ast_frdup does not duplicate integer-frame-data

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14080 
====================================================================== 
Reported By:                fnordian
Assigned To:                russell
====================================================================== 
Project:                    Asterisk
Issue ID:                   14080
Category:                   Core/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 164346 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-12-15 12:15 CST
Last Modified:              2008-12-15 18:38 CST
====================================================================== 
Summary:                    [patch] ast_frdup does not duplicate
integer-frame-data
Description: 
hi,
although frame.data is an enum and can be used as a ptr to a buffer and as
an integer, ast_frdup only copies frame-data, if it's a ptr (datalen > 0).

this causes e.g. that local_hangup hangup-causes don't get propagated.
====================================================================== 

---------------------------------------------------------------------- 
 (0096425) svnbot (reporter) - 2008-12-15 18:38
 http://bugs.digium.com/view.php?id=14080#c96425 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 164519

U   trunk/main/frame.c

------------------------------------------------------------------------
r164519 | russell | 2008-12-15 15:53:25 -0600 (Mon, 15 Dec 2008) | 7 lines

Make sure we handle a uint32_t payload in ast_frdup()

(closes issue http://bugs.digium.com/view.php?id=14080)
Reported by: fnordian
Patches:
      frame.patch uploaded by fnordian (license 110)

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

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

Repository: asterisk
Revision: 164520

_U  branches/1.6.0/

------------------------------------------------------------------------
r164520 | russell | 2008-12-15 15:54:47 -0600 (Mon, 15 Dec 2008) | 14
lines

Blocked revisions 164519 via svnmerge

........
r164519 | russell | 2008-12-15 15:53:30 -0600 (Mon, 15 Dec 2008) | 7 lines

Make sure we handle a uint32_t payload in ast_frdup()

(closes issue http://bugs.digium.com/view.php?id=14080)
Reported by: fnordian
Patches:
      frame.patch uploaded by fnordian (license 110)

........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-15 18:38 svnbot         Checkin                                      
2008-12-15 18:38 svnbot         Note Added: 0096425                          
======================================================================