[asterisk-commits] rmudgett: branch 12 r401030 - /branches/12/channels/dahdi/bridge_native_dahdi.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Tue Oct 15 15:25:37 2013
New Revision: 401030

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=401030
Log:
bridge_native_dahdi: Return channel join failure if could not make the channels compatible.

Modified:
    branches/12/channels/dahdi/bridge_native_dahdi.c

Modified: branches/12/channels/dahdi/bridge_native_dahdi.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/channels/dahdi/bridge_native_dahdi.c?view=diff&rev=401030&r1=401029&r2=401030
==============================================================================
--- branches/12/channels/dahdi/bridge_native_dahdi.c (original)
+++ branches/12/channels/dahdi/bridge_native_dahdi.c Tue Oct 15 15:25:37 2013
@@ -578,18 +578,17 @@
 	bridge_channel->tech_pvt = tech_pvt;
 	native_request_start(bridge);
 
+	/*
+	 * Make the channels compatible in case the native bridge did
+	 * not start for some reason and we need to fallback to 1-1
+	 * bridging.
+	 */
 	c0 = AST_LIST_FIRST(&bridge->channels)->chan;
 	c1 = AST_LIST_LAST(&bridge->channels)->chan;
-	if (c0 != c1) {
-		/*
-		 * Make the channels compatible in case the native bridge did
-		 * not start for some reason and we need to fallback to 1-1
-		 * bridging.
-		 */
-		ast_channel_make_compatible(c0, c1);
-	}
-
-	return 0;
+	if (c0 == c1) {
+		return 0;
+	}
+	return ast_channel_make_compatible(c0, c1);
 }
 
 /*!