Merged revisions 103904 via svnmerge from
authorMark Michelson <mmichelson@digium.com>
Wed, 20 Feb 2008 21:41:31 +0000 (21:41 +0000)
committerMark Michelson <mmichelson@digium.com>
Wed, 20 Feb 2008 21:41:31 +0000 (21:41 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r103904 | mmichelson | 2008-02-20 15:40:08 -0600 (Wed, 20 Feb 2008) | 6 lines

Fix a crash if the channel becomes NULL while attempting to lock it.

(closes issue #12039)
Reported by: danpwi

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@103908 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_local.c

index f03f133..c1c597b 100644 (file)
@@ -511,13 +511,15 @@ static int local_hangup(struct ast_channel *ast)
                const char *status = pbx_builtin_getvar_helper(p->chan, "DIALSTATUS");
                if ((status) && (p->owner)) {
                        /* Deadlock avoidance */
-                       while (ast_channel_trylock(p->owner)) {
+                       while (p->owner && ast_channel_trylock(p->owner)) {
                                ast_mutex_unlock(&p->lock);
                                usleep(1);
                                ast_mutex_lock(&p->lock);
                        }
-                       pbx_builtin_setvar_helper(p->owner, "CHANLOCALSTATUS", status);
-                       ast_channel_unlock(p->owner);
+                       if (p->owner) {
+                               pbx_builtin_setvar_helper(p->owner, "CHANLOCALSTATUS", status);
+                               ast_channel_unlock(p->owner);
+                       }
                }
                p->chan = NULL;
                ast_clear_flag(p, LOCAL_LAUNCHED_PBX);