From 889e0b00c2b3b4ecb8ab8116d6192ee7f3b37909 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Tue, 13 Jun 2023 12:58:26 +0300 Subject: [PATCH 2/2] subparse: Skip after the end of a valid closing tag instead of only skipping `<` This is a small optimization and avoids restarting the next parsing iteration on already accepted data. On its own it would also fix ZDI-CAN-20968 (see previous commit) but the previous commit independently is also a valid fix for it. Part-of: --- gst/subparse/gstsubparse.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gst/subparse/gstsubparse.c b/gst/subparse/gstsubparse.c index e8d3ecaef..9336419e1 100644 --- a/gst/subparse/gstsubparse.c +++ b/gst/subparse/gstsubparse.c @@ -827,6 +827,8 @@ subrip_fix_up_markup (gchar ** p_txt, gconstpointer allowed_tags_ptr) } else { --num_open_tags; g_ptr_array_remove_index (open_tags, num_open_tags); + cur = end_tag + 1; + continue; } } } -- 2.43.0