[asterisk-dev] q931.c: Fix DISCONNECT Progress Indicator ie handling. (libpri[1.4])

[Richard Mudgett]

Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/2426

Change subject: q931.c: Fix DISCONNECT Progress Indicator ie handling.
......................................................................

q931.c: Fix DISCONNECT Progress Indicator ie handling.

There are two scenarios that are exposed by DISCONNECT not initializing
the progress indicator value before processing the message when the
chan_dahdi.conf inbanddisconnect=yes option is set.

1) If a DISCONNECT comes in without a Progress Indicator ie and an earlier
message (such as SETUP-ACKNOWLEDGE or PROCEEDING) came in with the
indicator #8 (Inband audio present) then the DISCONNECT would not cause an
immediate hangup.  We would be letting the user hear the inband audio even
though there isn't any.

2) If a DISCONNECT message comes in with the indicator #8 (Inband audio
present) and then later the DISCONNECT message is repeated without a
Progress Indicator ie we would still ignore the second DISCONNECT to let
the user hear inband audio even though it likely isn't there anymore.

PRI-180 #close
Reported by: Alexandr Dranchuk

Change-Id: Ic88aafb45053146b5701d666e6212f7555573624
---
M q931.c
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/libpri refs/changes/26/2426/1

diff --git a/q931.c b/q931.c
index 69c210e..52afb3c 100644
--- a/q931.c
+++ b/q931.c
@@ -7381,6 +7381,8 @@
 		c->cause = -1;
 		c->causecode = -1;
 		c->causeloc = -1;
+		c->progress = -1;
+		c->progressmask = 0;
 		c->aoc_units = -1;
 		pri_schedule_del(ctrl, c->retranstimer);
 		c->retranstimer = 0;

-- 
To view, visit https://gerrit.asterisk.org/2426
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic88aafb45053146b5701d666e6212f7555573624
Gerrit-PatchSet: 1
Gerrit-Project: libpri
Gerrit-Branch: 1.4
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>