Attention is currently required from: pespin.
msuraev has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/30174 )

Change subject: osmux: Obey current batch_size restrictions when creating 
forged RTP packets to fill holes
......................................................................


Patch Set 3:

(1 comment)

File src/osmux_input.c:

https://gerrit.osmocom.org/c/libosmo-netif/+/30174/comment/3592aa48_82b84955
PS3, Line 448:  diff = cur_seq - last_seq;
Are there some restrictions on cur_seq and last_seq size? Otherwise if cur_seq 
== UINT16_MAX and last_seq == 1 for example than the diff won't fit into 
int16_t.



--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30174
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I4ea435bfb2490a375ad3e5068ee926e48b53cf5c
Gerrit-Change-Number: 30174
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Fri, 18 Nov 2022 15:52:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to