kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-11-03 20:38:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

core/frame: generate correct T.140 payload in ast_sendtext_data()

Browse Source 
ast_sendtext_data() would create an incorrect T.140 text frame which
length include the null terminator byte. It causes ultimately RTP
packets to be send with this trailing 0. The proposed fix just set the
correct length to the text frame

ASTERISK-28089
Reported by: Emmanuel BUU
Tested by: Emmanuel BUU

Change-Id: I7ab1b9ed1e21683b2b667ea0a59d9aba3c77dd96
This commit is contained in:
neutrino88
2018-10-03 17:51:49 -04:00
committed by George Joseph
parent 30717bafbf
commit 17f4e6ad4d
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
main/channel.c
View File

@@ -4678,7 +4678,8 @@ int ast_sendtext_data(struct ast_channel *chan, struct ast_msg_data *msg)
&& (ast_strlen_zero(content_type) || ast_begins_with(content_type, "text/"))
&& (ast_format_cap_has_type(ast_channel_nativeformats(chan), AST_MEDIA_TYPE_TEXT))) {
struct ast_frame f;
size_t body_len = strlen(body) + 1;
/* T.140 payload does not include the null byte terminator */
size_t body_len = strlen(body);
/* Process as T.140 text (moved here from ast_sendtext() */
memset(&f, 0, sizeof(f));