[Asterisk-code-review] app_stream_echo: Fix state of added streams. (asterisk[16])

Joshua Colp asteriskteam at digium.com
Thu Jun 18 03:51:09 CDT 2020


Joshua Colp has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/14567 )


Change subject: app_stream_echo: Fix state of added streams.
......................................................................

app_stream_echo: Fix state of added streams.

When stream support was added to Asterisk the stream state
was used inconsistently, resulting in odd behavior. This
was then standardized to be the state of a stream from the
perspective of Asterisk.

This change updates the StreamEcho dialplan application
to use the correct state, send only, since we are only
sending to the endpoint and not expecting them to send us
multiple video streams.

ASTERISK-28954

Change-Id: I35bfd533ef1184ffe62586b22bbd253c82872a56
---
M apps/app_stream_echo.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/67/14567/1

diff --git a/apps/app_stream_echo.c b/apps/app_stream_echo.c
index 717ed1c..669f92e 100644
--- a/apps/app_stream_echo.c
+++ b/apps/app_stream_echo.c
@@ -274,7 +274,7 @@
 			 * cloning the same stream.
 			 */
 			ast_stream_set_state(stream, n == num ?
-			     AST_STREAM_STATE_SENDRECV : AST_STREAM_STATE_RECVONLY);
+			     AST_STREAM_STATE_SENDRECV : AST_STREAM_STATE_SENDONLY);
 		} while (--n);
 	}
 

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I35bfd533ef1184ffe62586b22bbd253c82872a56
Gerrit-Change-Number: 14567
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200618/2dca9c61/attachment.html>


More information about the asterisk-code-review mailing list