[Home]

Summary:ASTERISK-13221: [patch] ast_frdup does not duplicate integer-frame-data
Reporter:Marcus Hunger (fnordian)Labels:
Date Opened:2008-12-15 12:15:37.000-0600Date Closed:2008-12-15 18:38:22.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) frame.patch
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.
Comments:By: Digium Subversion (svnbot) 2008-12-15 15:53:25.000-0600

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 ASTERISK-13221)
Reported by: fnordian
Patches:
     frame.patch uploaded by fnordian (license 110)

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

http://svn.digium.com/view/asterisk?view=rev&revision=164519
By: Digium Subversion (svnbot) 2008-12-15 18:38:22.000-0600

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 ASTERISK-13221)
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 ASTERISK-13221)
Reported by: fnordian
Patches:
     frame.patch uploaded by fnordian (license 110)

........

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

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