[asterisk-commits] file: trunk r82077 - /trunk/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: file
Date: Mon Sep 10 09:26:32 2007
New Revision: 82077

URL: http://svn.digium.com/view/asterisk?view=rev&rev=82077
Log:
(closes issue #10688)
Reported by: casper
Patches:
      chan_sip.c.82076.diff uploaded by casper (license 55)
Remove double check for zombie flag and optimize things a bit.

Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?view=diff&rev=82077&r1=82076&r2=82077
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Mon Sep 10 09:26:32 2007
@@ -4036,13 +4036,14 @@
 		p->owner = NULL;  /* Owner will be gone after we return, so take it away */
 		return 0;
 	}
-	if (ast_test_flag(ast, AST_FLAG_ZOMBIE) && p->refer)
-       		ast_debug(1, "SIP Transfer: Hanging up Zombie channel %s after transfer ... Call-ID: %s\n", ast->name, p->callid);
-	else
-		ast_debug(1, "Hangup call %s, SIP callid %s)\n", ast->name, p->callid);
-
-	if (ast_test_flag(ast, AST_FLAG_ZOMBIE)) 
-		ast_debug(1, "Hanging up zombie call. Be scared.\n");
+
+	if (ast_test_flag(ast, AST_FLAG_ZOMBIE)) {
+		if (p->refer)
+			ast_debug(1, "SIP Transfer: Hanging up Zombie channel %s after transfer ... Call-ID: %s\n", ast->name, p->callid);
+		else
+			ast_debug(1, "Hanging up zombie call. Be scared.\n");
+	} else
+		ast_debug(1, "Hangup call %s, SIP callid %s\n", ast->name, p->callid);
 
 	sip_pvt_lock(p);
 	if (ast_test_flag(&p->flags[0], SIP_INC_COUNT) || ast_test_flag(&p->flags[1], SIP_PAGE2_CALL_ONHOLD)) {