sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

res_rtp_asterisk: Ensure sufficient space for worst case NACK. 

ASTERISK-28790

Change-Id: I10df52f98b19ed62575f25dab36e82d136dccd99
This commit is contained in:
Joshua C. Colp 2020-03-25 06:38:53 -03:00 committed by Joshua Colp
parent ebddff3453
commit 7771a198d5
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
res/res_rtp_asterisk.c
View File

@ -7964,12 +7964,15 @@ static struct ast_frame *ast_rtp_read(struct ast_rtp_instance *instance, int rtc
int res = 0;
int ice;
int sr;
size_t data_size = AST_UUID_STR_LEN + 128 + (seqno - rtp->expectedrxseqno) / 17;
size_t data_size = AST_UUID_STR_LEN + 128 + (AST_VECTOR_SIZE(&rtp->missing_seqno) * 4);
RAII_VAR(unsigned char *, rtcpheader, NULL, ast_free_ptr);
RAII_VAR(struct ast_rtp_rtcp_report *, rtcp_report,
ast_rtp_rtcp_report_alloc(rtp->themssrc_valid ? 1 : 0),
ao2_cleanup);
/* Sufficient space for RTCP headers and report, SDES with CNAME, NACK header,
* and worst case 4 bytes per missing sequence number.
*/
rtcpheader = ast_malloc(sizeof(*rtcpheader) + data_size);
if (!rtcpheader) {
ast_debug(1, "Failed to allocate memory for NACK\n");