[Asterisk-code-review] Added RTT Support for PJSIP Added RTT Support for PJSIP (asterisk[16])

Bharat Ramaswamy-Nandakumar asteriskteam at digium.com
Fri Dec 13 19:04:08 CST 2019


Bharat Ramaswamy-Nandakumar has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/13446 )


Change subject: Added RTT Support for PJSIP Added RTT Support for PJSIP
......................................................................

Added RTT Support for PJSIP
Added RTT Support for PJSIP

Changes are:
1. chan_pjsip - added write_text function to the pjsip_tech driver
Added case for AST_FRAME_TEXT in chan_pjsip_write_stream
Added case to handle text in chan_pjsip_write

2. pjsip/dialplan_functions.c: added logic to handle media for TEXT

3.  res_pjsip.h: added variables to deal with endpoint media

4. channel.c: added  logic to add a stream_num for a frame

5.  frame.c: added code to handle and fix issues with red.

6. res_pjsip_sdp_rtp.c: added logic to setup SDP.
Added logic to handle tos and cos text
Added logic to RED for red enabled

7. res_pjsip_session.c: aded logic for max_text_streams

8. res_rtp_asterisk.c: Added logic to hanle Red and dealing with
a repeating RTP issue

9. pjsip_configuration.c: Added logic to handle text for PJSIP library

Issue ID: ASTERISK-28654

Reported-by: Bharat Ramaswamy Nandakumar(bharatram1)

Fixed by: Bharat Ramaswamy Nandakumar(bharatram1),
Seth Marks and Corey Wysong

Added fix to handle with the Build 13445 error in channel.c

Change-Id: I5915ef4835be741f551c389a5c63ece1821298df
---
M main/channel.c
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/46/13446/1

diff --git a/main/channel.c b/main/channel.c
index f8dcb76..09c222b 100644
--- a/main/channel.c
+++ b/main/channel.c
@@ -4698,8 +4698,7 @@
 		f.frametype = AST_FRAME_TEXT;
 		f.datalen = body_len;
 		f.mallocd = AST_MALLOCD_DATA;
-		struct ast_stream *default_streams = ast_channel_get_default_stream(chan, AST_MEDIA_TYPE_TEXT);
-		f.stream_num = ast_stream_get_position(default_streams);
+		f.stream_num = ast_stream_get_position(ast_channel_get_default_stream(chan, AST_MEDIA_TYPE_TEXT));
 		f.data.ptr = ast_strdup(body);
 		if (f.data.ptr) {
 			res = ast_channel_tech(chan)->write_text(chan, &f);

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I5915ef4835be741f551c389a5c63ece1821298df
Gerrit-Change-Number: 13446
Gerrit-PatchSet: 1
Gerrit-Owner: Bharat Ramaswamy-Nandakumar <bharat at indigital.net>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20191213/b31f0f82/attachment.html>


More information about the asterisk-code-review mailing list