From 1a7523608c6e8dc61589e18bde67870d6555aab5 Mon Sep 17 00:00:00 2001 From: mmichelson Date: Wed, 16 Jul 2008 19:37:42 +0000 Subject: Merged revisions 131357 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r131357 | mmichelson | 2008-07-16 14:37:08 -0500 (Wed, 16 Jul 2008) | 6 lines Apparently, "thread safety" is important, whatever that means. :P (Thanks Russell!) ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@131358 f38db490-d61c-443f-a65b-d21fe96a405b --- apps/app_queue.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'apps') diff --git a/apps/app_queue.c b/apps/app_queue.c index 8fc569846..dc9fa65a3 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -3798,7 +3798,7 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce * when the masquerade occurred. These other "ending" queue_log messages are unnecessary */ if (!attended_transfer_occurred(qe->chan)) { - struct ast_datastore *transfer_ds = ast_channel_datastore_find(qe->chan, &queue_transfer_info, NULL); + struct ast_datastore *transfer_ds; if (strcasecmp(oldcontext, qe->chan->context) || strcasecmp(oldexten, qe->chan->exten)) { ast_queue_log(queuename, qe->chan->uniqueid, member->membername, "TRANSFER", "%s|%s|%ld|%ld|%d", qe->chan->exten, qe->chan->context, (long) (callstart - qe->start), @@ -3813,12 +3813,13 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce (long) (callstart - qe->start), (long) (time(NULL) - callstart), qe->opos); send_agent_complete(qe, queuename, peer, member, callstart, vars, sizeof(vars), AGENT); } + ast_channel_lock(qe->chan); + transfer_ds = ast_channel_datastore_find(qe->chan, &queue_transfer_info, NULL); if (transfer_ds) { - ast_channel_lock(qe->chan); ast_channel_datastore_remove(qe->chan, transfer_ds); ast_channel_datastore_free(transfer_ds); - ast_channel_unlock(qe->chan); } + ast_channel_unlock(qe->chan); } if (bridge != AST_PBX_NO_HANGUP_PEER) -- cgit v1.2.3