----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4048/#review13477 -----------------------------------------------------------
Ship it! Ship It! - Matt Jordan On Oct. 6, 2014, 11:49 a.m., Corey Farrell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4048/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2014, 11:49 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-18923 > https://issues.asterisk.org/jira/browse/ASTERISK-18923 > > > Repository: Asterisk > > > Description > ------- > > fax_session_reserve adds a reference to the fax driver's module, then > fax_session_new adds another reference, but when the session is freed it only > removes a single reference to the module. > > This fixes it by unreferencing the module from fax_session_new when it is > based on a reserved session. > > > Diffs > ----- > > /branches/1.8/res/res_fax.c 424175 > > Diff: https://reviewboard.asterisk.org/r/4048/diff/ > > > Testing > ------- > > In testsuite against 11 verified that res_fax_spandsp now unloads in tests > that previously leaked references to the module. > > > 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
