From 3d904659ec3aa23565aa54690961048be7ec7f4c Mon Sep 17 00:00:00 2001 From: Corey Farrell Date: Wed, 22 Jun 2016 16:04:54 -0400 Subject: res_fax: Fix reference leak in fax_v21_session_new. fax_v21_session_new created a session details object but only released the allocation reference during error conditions. fax_session_new adds it's own reference to details if needed so the caller is always responsible for cleaning it's own reference. ASTERISK-26141 #close Change-Id: Ie7fc52a83b6596ce9ce2d5a2bd9f3e204f48fc88 --- res/res_fax.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/res/res_fax.c b/res/res_fax.c index 6282b13d7..ad6e2386c 100644 --- a/res/res_fax.c +++ b/res/res_fax.c @@ -2846,11 +2846,8 @@ static struct ast_fax_session *fax_v21_session_new (struct ast_channel *chan) { } v21_details->caps = AST_FAX_TECH_V21_DETECT; - if (!(v21_session = fax_session_new(v21_details, chan, NULL, NULL))) { - ao2_ref(v21_details, -1); - return NULL; - } - + v21_session = fax_session_new(v21_details, chan, NULL, NULL); + ao2_ref(v21_details, -1); return v21_session; } -- cgit v1.2.3