Merge pull request #20 from jaymell/wrapping-add-timestamp

use wrapping_add to prevent overflow panic -- fixes webrtc-rs/rtp#19

crates/rtp/src/packetizer/mod.rs

@@ -135,7 +135,7 @@ impl Packetizer for PacketizerImpl {
             i += 1;
         }
 
-        self.timestamp += samples;
+        self.timestamp.wrapping_add(samples);
 
         if l != 0 && self.abs_send_time != 0 {
             let st = if let Some(fn_time_gen) = &self.time_gen {

crates/rtp/src/packetizer/packetizer_test.rs

@@ -93,3 +93,16 @@ async fn test_packetizer_abs_send_time() -> Result<()> {
 
     Ok(())
 }
+
+#[tokio::test]
+async fn test_packetizer_timestamp_rollover_does_not_panic() -> Result<()> {
+    let g722 = Box::new(g7xx::G722Payloader {});
+    let seq = Box::new(new_random_sequencer());
+
+    let payload = Bytes::from_static(&[0; 128]);
+    let mut packetizer = new_packetizer(100, 98, 0x1234ABCD, g722, seq, 90000);
+
+    packetizer.packetize(&payload, u64::MAX as _).await?;
+
+    Ok(())
+}