From: Sebastian Dröge <sebastian@centricular.com>
Date: Fri, 2 Sep 2022 14:21:43 +0000 (+0300)
Subject: rtpjitterbuffer: Add test for crash caused by removing timers twice
X-Git-Tag: 1.22.0~1000
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cc454f0fc3840fc3d9519ea1d5b192bb13e354b4;p=platform%2Fupstream%2Fgstreamer.git

rtpjitterbuffer: Add test for crash caused by removing timers twice

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2973>
---

diff --git a/subprojects/gst-plugins-good/tests/check/elements/rtpjitterbuffer.c b/subprojects/gst-plugins-good/tests/check/elements/rtpjitterbuffer.c
index 2400fd6..1712399 100644
--- a/subprojects/gst-plugins-good/tests/check/elements/rtpjitterbuffer.c
+++ b/subprojects/gst-plugins-good/tests/check/elements/rtpjitterbuffer.c
@@ -3371,6 +3371,28 @@ GST_START_TEST (test_reset_timers_does_not_stall)
 
 GST_END_TEST;
 
+GST_START_TEST (test_reset_timers_does_not_stall_2)
+{
+  GstHarness *h = gst_harness_new ("rtpjitterbuffer");
+  BufferArrayCtx bufs[] = {
+    /* *INDENT-OFF* */
+    {278, 21920, FALSE,  31695},
+    { 37,  5920, FALSE,  89911},
+    {173, 13600, FALSE, 108078},
+    { 30, 27200, FALSE, 190920},
+    {-20, 43840,  TRUE, 150552},
+    { 42,  4480, FALSE, 131498},
+    /* *INDENT-ON* */
+  };
+
+  g_object_set (h->element, "latency", 200,
+      "do-retransmission", TRUE, "do-lost", TRUE, NULL);
+  fail_unless (check_for_stall (h, bufs, G_N_ELEMENTS (bufs)));
+  gst_harness_teardown (h);
+}
+
+GST_END_TEST;
+
 GST_START_TEST (test_multiple_lost_do_not_stall)
 {
   GstHarness *h = gst_harness_new ("rtpjitterbuffer");
@@ -3508,6 +3530,7 @@ rtpjitterbuffer_suite (void)
   tcase_add_test (tc_chain, test_drop_messages_interval);
 
   tcase_add_test (tc_chain, test_reset_timers_does_not_stall);
+  tcase_add_test (tc_chain, test_reset_timers_does_not_stall_2);
   tcase_add_test (tc_chain, test_multiple_lost_do_not_stall);
   tcase_add_test (tc_chain, test_reset_using_rtx_packets_does_not_stall);
   tcase_add_test (tc_chain, test_gap_using_rtx_does_not_stall);