[Asterisk-code-review] say: Abort play loop if caller hangs up. (asterisk[master])

[N A]

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


Change subject: say: Abort play loop if caller hangs up.
......................................................................

say: Abort play loop if caller hangs up.

If the caller has hung up, break out of the play
loop so we don't try to play remaining files
and fail to do so.

ASTERISK-30075 #close

Change-Id: I55e85be28ee90b48c0fe4ce20ac136a7dbb49f14
---
M main/say.c
1 file changed, 4 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/75/18575/1

diff --git a/main/say.c b/main/say.c
index df9462a..39b50c6 100644
--- a/main/say.c
+++ b/main/say.c
@@ -200,10 +200,12 @@
 			if (res > 0) {
 				/* We were interrupted by a digit */
 				ast_stopstream(chan);
-				ast_free(filenames);
-				return res;
+				break;
 			}
 		}
+		if (res < 0) {
+			break;
+		}
 		ast_stopstream(chan);
 	}
 

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I55e85be28ee90b48c0fe4ce20ac136a7dbb49f14
Gerrit-Change-Number: 18575
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220521/7569e728/attachment.html>