----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3751/#review12572 -----------------------------------------------------------
Ship it! Ship It! - opticron On July 11, 2014, 5 p.m., Corey Farrell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3751/ > ----------------------------------------------------------- > > (Updated July 11, 2014, 5 p.m.) > > > Review request for Asterisk Developers and Matt Jordan. > > > Repository: Asterisk > > > Description > ------- > > * chan_sip: sip_new unref's fmt twice. The unref that remains is at 8186. > The check for if (fmt) should no longer be needed due addition of > ast_format_none. This check was ineffective anyways since fmt was used past > the if statement. > > * frame.c: ensure fr->mallocd is cleared if we are not freeing fr. > > > Diffs > ----- > > /team/group/media_formats-reviewed-trunk/main/frame.c 418416 > /team/group/media_formats-reviewed-trunk/channels/chan_sip.c 418416 > > Diff: https://reviewboard.asterisk.org/r/3751/diff/ > > > Testing > ------- > > > Thanks, > > Corey Farrell > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
