diff options
author | Jan Schmidt <jan@centricular.com> | 2020-05-28 17:33:43 +1000 |
---|---|---|
committer | GStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org> | 2020-05-28 10:58:02 +0000 |
commit | 0e578b1096e9ccca167e4cccd6c160a615223fb6 (patch) | |
tree | ea10aafe153f0088a966d2e8763ca38985a0106d /tests | |
parent | 67ae885d4c4b6e510c38882f0edb0800d8ed0946 (diff) | |
download | gstreamer-plugins-bad-0e578b1096e9ccca167e4cccd6c160a615223fb6.tar.gz |
avtp: Fix some leaks in the tests
Fix valgrind errors that area showing on the CI now
that AVTP elements are built.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1305>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/check/elements/avtpcrfbase.c | 24 | ||||
-rw-r--r-- | tests/check/elements/avtpcrfcheck.c | 3 | ||||
-rw-r--r-- | tests/check/elements/avtpcrfsync.c | 8 |
3 files changed, 35 insertions, 0 deletions
diff --git a/tests/check/elements/avtpcrfbase.c b/tests/check/elements/avtpcrfbase.c index 0f676f57f..d7430162d 100644 --- a/tests/check/elements/avtpcrfbase.c +++ b/tests/check/elements/avtpcrfbase.c @@ -68,6 +68,7 @@ GST_START_TEST (test_validate_crf_pdu_success) fail_unless (thread_data->timestamp_interval == 160); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -101,6 +102,7 @@ GST_START_TEST (test_validate_crf_pdu_multiple_packets_success) g_free (crf_pdu1); g_free (crf_pdu2); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -122,6 +124,7 @@ GST_START_TEST (test_validate_crf_pdu_wrong_subtype) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -142,6 +145,7 @@ GST_START_TEST (test_validate_crf_pdu_streamid_invalid) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -160,6 +164,7 @@ GST_START_TEST (test_validate_crf_pdu_streamid_different) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -178,6 +183,7 @@ GST_START_TEST (test_validate_crf_pdu_data_len_too_long) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -198,6 +204,7 @@ GST_START_TEST (test_validate_crf_pdu_timestamp_interval_zero) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -218,6 +225,7 @@ GST_START_TEST (test_validate_crf_pdu_base_freq_zero) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -238,6 +246,7 @@ GST_START_TEST (test_validate_crf_pdu_pull_invalid) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -258,6 +267,7 @@ GST_START_TEST (test_validate_crf_pdu_type_invalid) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -278,6 +288,7 @@ GST_START_TEST (test_validate_crf_pdu_data_len_invalid) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -302,6 +313,7 @@ GST_START_TEST (test_validate_crf_pdu_timestamp_interval_mismatch) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -326,6 +338,7 @@ GST_START_TEST (test_validate_crf_pdu_base_freq_mismatch) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -350,6 +363,7 @@ GST_START_TEST (test_validate_crf_pdu_pull_mismatch) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -374,6 +388,7 @@ GST_START_TEST (test_validate_crf_pdu_type_mismatch) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -401,6 +416,7 @@ GST_START_TEST (test_validate_crf_pdu_data_len_mismatch) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -428,6 +444,7 @@ GST_START_TEST (test_validate_crf_pdu_tstamps_not_monotonic) fail_unless (ret == FALSE); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -457,6 +474,7 @@ GST_START_TEST (test_gst_base_freq_multiplier) ret = get_base_freq_multiplier (avtpcrfbase, 6); fail_unless_equals_float (ret, -1); + gst_object_unref (avtpcrfbase); } @@ -499,6 +517,7 @@ GST_START_TEST (test_calculate_average_period_multiple_crf_tstamps) fail_unless_equals_uint64 (thread_data->past_periods[2], 20833); fail_unless_equals_uint64 (thread_data->current_ts, 1000); + gst_object_unref (avtpcrfbase); g_free (crf_pdu); } @@ -531,6 +550,7 @@ GST_START_TEST fail_unless_equals_uint64 (thread_data->current_ts, 18446744073709501615ULL); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -566,6 +586,7 @@ GST_START_TEST (test_calculate_average_period_single_crf_tstamp) fail_unless_equals_uint64 (thread_data->current_ts, 21833); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -605,6 +626,7 @@ GST_START_TEST (test_calculate_average_period_single_crf_tstamp_init) g_free (crf_pdu1); g_free (crf_pdu2); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -640,6 +662,7 @@ GST_START_TEST (test_calculate_average_period_single_crf_tstamp_64_bit_overflow) fail_unless_equals_uint64 (thread_data->current_ts, 20833); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; @@ -676,6 +699,7 @@ GST_START_TEST (test_calculate_average_period_single_crf_tstamp_seq_num_skip) fail_unless_equals_uint64 (thread_data->current_ts, 21833); g_free (crf_pdu); + gst_object_unref (avtpcrfbase); } GST_END_TEST; diff --git a/tests/check/elements/avtpcrfcheck.c b/tests/check/elements/avtpcrfcheck.c index efe4a2060..a967ea020 100644 --- a/tests/check/elements/avtpcrfcheck.c +++ b/tests/check/elements/avtpcrfcheck.c @@ -176,6 +176,7 @@ GST_START_TEST (test_crf_cvf_data) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfcheck"); avtpcrfbase->thread_data.average_period = 3300; avtpcrfbase->thread_data.current_ts = 110000; + gst_object_unref (avtpcrfbase); test_crf_tstamps (h, buf, 110000, 109204, 1); test_crf_tstamps (h, buf, 113600, 119400, 2); @@ -199,6 +200,7 @@ GST_START_TEST (test_crf_aaf_data) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfcheck"); avtpcrfbase->thread_data.average_period = 3300; avtpcrfbase->thread_data.current_ts = 110000; + gst_object_unref (avtpcrfbase); test_crf_tstamps (h, buf, 113300, 0, 1); test_crf_tstamps (h, buf, 112900, 0, 2); @@ -221,6 +223,7 @@ GST_START_TEST (test_crf_period_zero) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfcheck"); avtpcrfbase->thread_data.average_period = 0.0; avtpcrfbase->thread_data.current_ts = 110; + gst_object_unref (avtpcrfbase); test_crf_tstamps (h, buf, 112, 110, 1); diff --git a/tests/check/elements/avtpcrfsync.c b/tests/check/elements/avtpcrfsync.c index 6990d6c77..9f6e92571 100644 --- a/tests/check/elements/avtpcrfsync.c +++ b/tests/check/elements/avtpcrfsync.c @@ -169,6 +169,8 @@ test_crf_tstamps (GstHarness * h, GstBuffer * buf, struct buffer_tstamps *orig, bufout = gst_harness_push_and_pull (h, buf); validate_tstamps (avtpcrfbase, bufout, expected); + + gst_object_unref (avtpcrfbase); } GST_START_TEST (test_properties) @@ -258,6 +260,7 @@ GST_START_TEST (test_crf_cvf_data) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfsync"); avtpcrfbase->thread_data.average_period = 3300; avtpcrfbase->thread_data.current_ts = 110000; + gst_object_unref (avtpcrfbase); orig = (struct buffer_tstamps) { .buf_pts = 103000,.buf_dts = 100000,.avtp_ts = 110000,.h264_ts = 108000}; @@ -280,6 +283,7 @@ GST_START_TEST (test_crf_cvf_data) 113300,.h264_ts = 119900}; test_crf_tstamps (h, buf, &orig, &expected); + gst_buffer_unref (buf); gst_harness_teardown (h); } @@ -298,6 +302,7 @@ GST_START_TEST (test_crf_aaf_data) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfsync"); avtpcrfbase->thread_data.average_period = 3300; avtpcrfbase->thread_data.current_ts = 110000; + gst_object_unref (avtpcrfbase); orig = (struct buffer_tstamps) { .buf_pts = 108000,.buf_dts = 0,.avtp_ts = 110000,.h264_ts = 0}; @@ -311,6 +316,7 @@ GST_START_TEST (test_crf_aaf_data) .buf_pts = 110300,.buf_dts = 0,.avtp_ts = 113300,.h264_ts = 0}; test_crf_tstamps (h, buf, &orig, &expected); + gst_buffer_unref (buf); gst_harness_teardown (h); } @@ -329,6 +335,7 @@ GST_START_TEST (test_crf_period_zero) avtpcrfbase = (GstAvtpCrfBase *) gst_harness_find_element (h, "avtpcrfsync"); avtpcrfbase->thread_data.average_period = 0.0; avtpcrfbase->thread_data.current_ts = 110; + gst_object_unref (avtpcrfbase); orig = (struct buffer_tstamps) { .buf_pts = 100,.buf_dts = 105,.avtp_ts = 112,.h264_ts = 110}; @@ -336,6 +343,7 @@ GST_START_TEST (test_crf_period_zero) .buf_pts = 100,.buf_dts = 105,.avtp_ts = 112,.h264_ts = 110}; test_crf_tstamps (h, buf, &orig, &expected); + gst_buffer_unref (buf); gst_harness_teardown (h); } |