diff options
author | murf <murf@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-18 23:41:33 +0000 |
---|---|---|
committer | murf <murf@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-18 23:41:33 +0000 |
commit | d40eb60d1b4363dcb77f240e7490f42bfc46238d (patch) | |
tree | eaeb5fba01417d32279b5f92338706e50211d605 | |
parent | 2bbfcd96512795e140e8abcb883b40858eaa3aa3 (diff) |
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/trunk@143559 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_sip.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index ea1a0e327..7b1f0b13f 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -16259,12 +16259,12 @@ static void *sip_park_thread(void *stuff) transferee = d->chan1; transferer = d->chan2; copy_request(&req, &d->req); - if (d->req.data) - ast_free(d->req.data); - ast_free(d); if (!transferee || !transferer) { ast_log(LOG_ERROR, "Missing channels for parking! Transferer %s Transferee %s\n", transferer ? "<available>" : "<missing>", transferee ? "<available>" : "<missing>" ); + 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); @@ -16274,6 +16274,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); @@ -16307,6 +16310,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; } |