From d935a3f2b7540b843f9564e0e634246303617f08 Mon Sep 17 00:00:00 2001 From: murf Date: Thu, 18 Sep 2008 23:50:25 +0000 Subject: Merged revisions 143559 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r143559 | murf | 2008-09-18 17:41:33 -0600 (Thu, 18 Sep 2008) | 9 lines Merged revisions 143534 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r143534 | murf | 2008-09-18 16:11:51 -0600 (Thu, 18 Sep 2008) | 1 line A micro-fix, in sip_park_thread, where d is freed before the func is done using it. ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@143560 f38db490-d61c-443f-a65b-d21fe96a405b --- channels/chan_sip.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'channels') diff --git a/channels/chan_sip.c b/channels/chan_sip.c index a78489446..31a3a6604 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -15710,10 +15710,12 @@ static void *sip_park_thread(void *stuff) transferee = d->chan1; transferer = d->chan2; copy_request(&req, &d->req); - ast_free(d); if (!transferee || !transferer) { ast_log(LOG_ERROR, "Missing channels for parking! Transferer %s Transferee %s\n", transferer ? "" : "", transferee ? "" : "" ); + if (d->req.data) + ast_free(d->req.data); + free(d); return NULL; } ast_debug(4, "SIP Park: Transferer channel %s, Transferee %s\n", transferer->name, transferee->name); @@ -15723,6 +15725,9 @@ static void *sip_park_thread(void *stuff) ast_log(LOG_WARNING, "Masquerade failed.\n"); transmit_response(transferer->tech_pvt, "503 Internal error", &req); ast_channel_unlock(transferee); + if (d->req.data) + ast_free(d->req.data); + free(d); return NULL; } ast_channel_unlock(transferee); @@ -15756,6 +15761,9 @@ static void *sip_park_thread(void *stuff) ast_debug(1, "SIP Call parked failed \n"); /* Do not hangup call */ } + if (d->req.data) + ast_free(d->req.data); + free(d); return NULL; } -- cgit v1.2.3