res_rtp_asterisk.c: Fix signed mismatch that leads to overflow

ASTERISK-29205 #close

Change-Id: Ib7aa65644e8df76e2378d7613ee7cf751b9d0bea
changes/87/15487/2
Sean Bright 6 years ago committed by George Joseph
parent 23e96e6e17
commit 07eddbd56a

@ -6771,7 +6771,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;
}
@ -6805,7 +6805,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