diff options
| author | jpeeler <jpeeler@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-01-06 19:09:08 +0000 |
|---|---|---|
| committer | jpeeler <jpeeler@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-01-06 19:09:08 +0000 |
| commit | d0156cd769bf49b0387bd099b2ac8d96c58fd389 (patch) | |
| tree | 284193a5b7959a19818a3d772f72ee213db3eb3e | |
| parent | 355bf97fd7bc3b319ea6597402e16738a208ba36 (diff) | |
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)
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@238137 f38db490-d61c-443f-a65b-d21fe96a405b
| -rw-r--r-- | main/features.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/main/features.c b/main/features.c index e6d0ceaad..cdedb9664 100644 --- a/main/features.c +++ b/main/features.c @@ -4530,9 +4530,7 @@ static int bridge_exec(struct ast_channel *chan, void *data) 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" |