[Asterisk-code-review] sig_analog: Fix no timeout duration. (asterisk[master])

N A asteriskteam at digium.com
Thu Dec 1 05:56:26 CST 2022


N A has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/19613 )


Change subject: sig_analog: Fix no timeout duration.
......................................................................

sig_analog: Fix no timeout duration.

ASTERISK_28702 previously attempted to fix an
issue with flash hook hold timing out after
just under 17 minutes, when it should have never
been timing out. It fixed this by changing 999999
to INT_MAX, but it did so in chan_dahdi, which
is the wrong place since ss_thread is now in
sig_analog and the one in chan_dahdi is mostly
dead code.

This fixes this by porting the fix to sig_analog.

ASTERISK-30336 #close

Change-Id: I05eb69cc0b5319d357842a70bd26ef64d145cb15
---
M channels/sig_analog.c
1 file changed, 23 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/13/19613/1

diff --git a/channels/sig_analog.c b/channels/sig_analog.c
index ccc14a8..b694a96 100644
--- a/channels/sig_analog.c
+++ b/channels/sig_analog.c
@@ -2138,7 +2138,7 @@
 		/* If starting a threeway call, never timeout on the first digit so someone
 		   can use flash-hook as a "hold" feature */
 		if (p->subs[ANALOG_SUB_THREEWAY].owner) {
-			timeout = 999999;
+			timeout = INT_MAX;
 		}
 		while (len < AST_MAX_EXTENSION-1) {
 			int is_exten_parking = 0;

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/19613
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I05eb69cc0b5319d357842a70bd26ef64d145cb15
Gerrit-Change-Number: 19613
Gerrit-PatchSet: 1
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221201/10eaca48/attachment.html>


More information about the asterisk-code-review mailing list