----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3740/#review12549 -----------------------------------------------------------
/team/group/media_formats-reviewed-trunk/channels/chan_dahdi.c <https://reviewboard.asterisk.org/r/3740/#comment22837> I wonder why __unload_module() isn't called instead since we've allocated a bunch of other stuff. - rmudgett On July 10, 2014, 2:41 p.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3740/ > ----------------------------------------------------------- > > (Updated July 10, 2014, 2:41 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > chan_dahdi was really kind of a bonus, my goal here was to make sure a clean > start up + shutdown of format and frame stuff in the core without any modules > loaded didn't leak memory. I think this is the last container not getting > destroyed on shutdown. > > > Diffs > ----- > > /team/group/media_formats-reviewed-trunk/main/format.c 418348 > /team/group/media_formats-reviewed-trunk/channels/chan_dahdi.c 418349 > > Diff: https://reviewboard.asterisk.org/r/3740/diff/ > > > Testing > ------- > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- 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
