res_rtp_asterisk.c: Fix signed mismatch that leads to overflow

ASTERISK-29205 #close

Change-Id: Ib7aa65644e8df76e2378d7613ee7cf751b9d0bea
changes/88/15488/2
Sean Bright 6 years ago committed by George Joseph
parent 9c0e86b8d7
commit 8fb9da2fd9

@ -6871,7 +6871,7 @@ static int rtp_transport_wide_cc_feedback_produce(const void *data)
/* If there is no more room left for storing packets stop now, we leave 20
* extra bits at the end just in case.
*/
if ((sizeof(bdata) - (packet_len + delta_len + 20)) < 0) {
if (packet_len + delta_len + 20 > sizeof(bdata)) {
res = -1;
break;
}
@ -6905,7 +6905,7 @@ static int rtp_transport_wide_cc_feedback_produce(const void *data)
previous_packet = statistics;
/* If there is no more room left in the packet stop handling of any subsequent packets */
if ((sizeof(bdata) - (packet_len + delta_len + 20)) < 0) {
if (packet_len + delta_len + 20 > sizeof(bdata)) {
break;
}
}

Loading…
Cancel
Save