[asterisk-commits] jpeeler: branch 1.6.2 r238137 - in /branches/1.6.2: ./ main/features.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Jan 6 13:09:12 CST 2010


Author: jpeeler
Date: Wed Jan  6 13:09:08 2010
New Revision: 238137

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=238137
Log:
Merged revisions 238134 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r238134 | jpeeler | 2010-01-06 13:05:06 -0600 (Wed, 06 Jan 2010) | 10 lines
  
  Fix channel name comparison for bridge application.
  
  The channel name comparison was not comparing the whole string and therefore
  if one channel name was a substring of the other, the bridge would fail.
  
  (closes issue #16528)
  Reported by: telecos82
  Patches: 
        res_features_r236843.diff uploaded by telecos82 (license 687)
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/main/features.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/main/features.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/main/features.c?view=diff&rev=238137&r1=238136&r2=238137
==============================================================================
--- branches/1.6.2/main/features.c (original)
+++ branches/1.6.2/main/features.c Wed Jan  6 13:09:08 2010
@@ -4530,9 +4530,7 @@
 		ast_app_parse_options(bridge_exec_options, &opts, NULL, args.options);
 
 	/* avoid bridge with ourselves */
-	if (!strncmp(chan->name, args.dest_chan, 
-		strlen(chan->name) < strlen(args.dest_chan) ? 
-		strlen(chan->name) : strlen(args.dest_chan))) {
+	if (!strcmp(chan->name, args.dest_chan)) {
 		ast_log(LOG_WARNING, "Unable to bridge channel %s with itself\n", chan->name);
 		manager_event(EVENT_FLAG_CALL, "BridgeExec",
 					"Response: Failed\r\n"




More information about the asterisk-commits mailing list