[Home]

Summary:ASTERISK-26596: Placing call on hold temporarily locks up set
Reporter:Igor Goncharovsky (igorg)Labels:patch
Date Opened:2016-11-14 22:07:18.000-0600Date Closed:2018-01-22 16:18:02.000-0600
Priority:MajorRegression?
Status:Closed/CompleteComponents:Channels/chan_unistim
Versions:11.24.1 13.12.1 14.1.1 Frequency of
Occurrence
Constant
Related
Issues:
is the original version of this clone:ASTERISK-26565 chan_unistim on 11, 13, 14 placing call on hold temporarily locks up set
Environment:Debian Linux x86 Intel, Debian Linux ARMh/ARMlAttachments:( 0) ASTERISK-26565-v1.diff
Description:With a Nortel i2002 (also tested on unknown i2001) places or receives a call, and an audio path is brought up (as in, audio is transmitted and/or received from an application or another channel), eg: a call is established, pressing the hold button properly sets up a bridge of MOH to the other side (they hear music), a caller in this example. At this point, the Nortel set is locked. Icons indicate a call is on hold, but no further buttons effect any function. Looking at debug messages, when the switchhook is pressed, even repeatedly, a message is generated on every press:
"WARNING[2075] chan_unistim.c: Close call without sub"
Interestingly, when the other end of the call hangs-up, the icon on the Nortel set indicates such, but remains locked. The UDP/RTP ports are still held at this point. But, placing a call to the same locked-up set, from any functioning channel, unlocks the set, and CLID etc are there, and ringing away. Answering the call, (on the Nortel set) it's all back to normal, and after parties hang up, both/all UDP/RTP ports are properly released.
(Note: ports being released is dependent on the patch from ASTERISK-26561 which fixed this on 7 November, 2016.) This is repeatable on three Nortel sets, acoss releases.
Comments:By: Asterisk Team (asteriskteam) 2016-11-14 22:07:19.223-0600

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Mat Jaggard (mjaggard) 2017-10-30 15:39:04.133-0500

This seems to partially fix the issue but the phones now only work fully for about a day. I don't have a clue what the remaining problem is but this patch did help.

By: Mat Jaggard (mjaggard) 2017-10-30 15:39:32.637-0500

Will this go into master?

By: Igor Goncharovsky (igorg) 2017-10-30 23:36:24.430-0500

Yes, this patch is only partial solution. I need to finish it and the it'll be committed to master, 13 and 15 branches.

By: Igor Goncharovsky (igorg) 2018-01-09 00:00:29.845-0600

Patch uploaded to Gerrit for review

By: Mat Jaggard (mjaggard) 2018-01-14 05:16:37.909-0600

Do you know when this will be put into a release branch?

By: Igor Goncharovsky (igorg) 2018-01-14 12:47:36.171-0600

In next release of currently supported branches after patch acceptance. You can help with it by testing and reporting results. Currently I work on patch update and submitting it on reviewboard.

By: Friendly Automation (friendly-automation) 2018-01-22 16:18:03.035-0600

Change 7880 merged by Jenkins2:
chan_unistim: Fix hold function ability to lock/crash asterisk

[https://gerrit.asterisk.org/7880|https://gerrit.asterisk.org/7880]

By: Friendly Automation (friendly-automation) 2018-01-22 16:33:46.222-0600

Change 7881 merged by Jenkins2:
chan_unistim: Fix hold function ability to lock/crash asterisk

[https://gerrit.asterisk.org/7881|https://gerrit.asterisk.org/7881]

By: Friendly Automation (friendly-automation) 2018-01-22 16:42:59.144-0600

Change 7882 merged by Jenkins2:
chan_unistim: Fix hold function ability to lock/crash asterisk

[https://gerrit.asterisk.org/7882|https://gerrit.asterisk.org/7882]