[asterisk-commits] rmudgett: branch 1.8 r371860 - /branches/1.8/apps/app_dial.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Wed Aug 29 13:22:24 2012
New Revision: 371860

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=371860
Log:
Fix hangup cause passthrough regression.

The v1.8 -r369258 change to fix the F and F(x) action logic introduced a
regression in passing the hangup cause from the called channel to the
caller channel.

(closes issue ASTERISK-20287)
Reported by: Konstantin Suvorov
Patches:
      app_dial_hangupcause.patch (license #6421) patch uploaded by Konstantin Suvorov (modified)
Tested by: rmudgett

Modified:
    branches/1.8/apps/app_dial.c

Modified: branches/1.8/apps/app_dial.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/apps/app_dial.c?view=diff&rev=371860&r1=371859&r2=371860
==============================================================================
--- branches/1.8/apps/app_dial.c (original)
+++ branches/1.8/apps/app_dial.c Wed Aug 29 13:22:24 2012
@@ -2994,9 +2994,9 @@
 					/* The peer is now running its own PBX. */
 					goto out;
 				}
-			} else {
-				chan->hangupcause = peer->hangupcause;
-			}
+			}
+		} else if (!ast_check_hangup(chan)) {
+			chan->hangupcause = peer->hangupcause;
 		}
 		ast_hangup(peer);
 	}