Author: coreyfarrell Date: Sun Nov 9 02:01:18 2014 New Revision: 427644 URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=427644 Log: manager: Fix HTTP connection reference leaks.
Fix reference leak that happens if (session && !blastaway). ASTERISK-24505 #close Reported by: Corey Farrell Review: https://reviewboard.asterisk.org/r/4153/ ........ Merged revisions 427641 from http://svn.asterisk.org/svn/asterisk/branches/11 ........ Merged revisions 427642 from http://svn.asterisk.org/svn/asterisk/branches/12 ........ Merged revisions 427643 from http://svn.asterisk.org/svn/asterisk/branches/13 Modified: trunk/ (props changed) trunk/main/manager.c Propchange: trunk/ ------------------------------------------------------------------------------ Binary property 'branch-13-merged' - no diff available. Modified: trunk/main/manager.c URL: http://svnview.digium.com/svn/asterisk/trunk/main/manager.c?view=diff&rev=427644&r1=427643&r2=427644 ============================================================================== --- trunk/main/manager.c (original) +++ trunk/main/manager.c Sun Nov 9 02:01:18 2014 @@ -7515,11 +7515,16 @@ ast_free(http_header); ast_free(out); - if (session && blastaway) { - session_destroy(session); - } else if (session && session->f) { - fclose(session->f); - session->f = NULL; + if (session) { + if (blastaway) { + session_destroy(session); + } else { + if (session->f) { + fclose(session->f); + session->f = NULL; + } + unref_mansession(session); + } } return 0; -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- svn-commits mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/svn-commits
