[Asterisk-code-review] frame.c: Copy the whole subclass in ast frdup(). (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Wed Apr 6 11:11:48 CDT 2016


Hello Anonymous Coward #1000019,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/2538

to look at the new patch set (#2).

Change subject: frame.c: Copy the whole subclass in ast_frdup().
......................................................................

frame.c: Copy the whole subclass in ast_frdup().

The problem is ast_frdup() does not copy whole frame.subclass for voice,
video and image frames, only the format is copied.  For video frames, the
subclass structure contains the .frame_ending flag used to put the RTP
marker where it needs to be.

ASTERISK-25894 #close

Change-Id: I812ca90e84ed5d4f473b997d0dd0d3c5a915fe33
---
M main/frame.c
1 file changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/38/2538/2
-- 
To view, visit https://gerrit.asterisk.org/2538
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I812ca90e84ed5d4f473b997d0dd0d3c5a915fe33
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Jacek Konieczny <jkonieczny at eggsoft.pl>
Gerrit-Reviewer: Anonymous Coward #1000019



More information about the asterisk-code-review mailing list