[asterisk-commits] igorg: branch 13 r425669 - in /branches/13: ./ channels/chan_unistim.c

[SVN commits to the Asterisk project]

Author: igorg
Date: Thu Oct 16 01:11:28 2014
New Revision: 425669

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=425669
Log:

Fix loss of voice after second call drops (on a second line) in case using multiple lines on unistim phones. There is regression was introduced in r391379.

Reported by: Rustam Khankishyiev
(closes issue ASTERISK-23846)
........

Merged revisions 425667 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 425668 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    branches/13/   (props changed)
    branches/13/channels/chan_unistim.c

Propchange: branches/13/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: branches/13/channels/chan_unistim.c
URL: http://svnview.digium.com/svn/asterisk/branches/13/channels/chan_unistim.c?view=diff&rev=425669&r1=425668&r2=425669
==============================================================================
--- branches/13/channels/chan_unistim.c (original)
+++ branches/13/channels/chan_unistim.c Thu Oct 16 01:11:28 2014
@@ -4953,17 +4953,19 @@
 	} else if (sub->subtype == SUB_RING) {
 		send_no_ring(s);
 		for (i = 0; i < FAVNUM; i++) {
-			if (!soft_key_visible(s->device, i))
+			if (!soft_key_visible(s->device, i)) {
 				continue;
-			if (d->ssub[i] != sub)
+			}
+			if (d->ssub[i] != sub) {
+				if (d->ssub[i] != NULL) { /* Found other subchannel active other then hangup'ed one */
+					end_call = 0;
+				}
 				continue;
+			}
 			if (is_key_line(d, i) && !strcmp(l->name, d->sline[i]->name)) {
 				send_favorite_short(i, FAV_LINE_ICON, s);
 				d->ssub[i] = NULL;
 				continue;
-			}
-			if (d->ssub[i] != NULL) { /* Found other subchannel active other then hangup'ed one */
-				end_call = 0;
 			}
 		}
 	}