diff options
author | Corey Farrell <git@cfware.com> | 2014-11-09 08:01:18 +0000 |
---|---|---|
committer | Corey Farrell <git@cfware.com> | 2014-11-09 08:01:18 +0000 |
commit | 21c41e4542290747bf90d8086f60c39030367ff9 (patch) | |
tree | 1b6016df1999829a8f2def1bb13399096b43e304 /main/manager.c | |
parent | f4392c4b6dc96764a6bd7f6d5c5612b98a5c6fa4 (diff) |
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
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@427644 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/manager.c')
-rw-r--r-- | main/manager.c | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/main/manager.c b/main/manager.c index ceee37f31..6fe12c50d 100644 --- a/main/manager.c +++ b/main/manager.c @@ -7515,11 +7515,16 @@ generic_callback_out: 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; |