diff options
author | He Junyan <junyan.he@intel.com> | 2023-02-12 16:11:34 +0800 |
---|---|---|
committer | Tim-Philipp Müller <tim@centricular.com> | 2023-07-18 12:30:57 +0200 |
commit | 107553843c27ef4386d4f91232088cfd5efad443 (patch) | |
tree | 1e48828a55bf0e7fb3ff8a9fd1e217bf345d5949 | |
parent | 1e7c085791dc60dbeb4872117a0366cbf95deb39 (diff) |
va: jpegdecoder: Do not check SOS state when parsing DRI marker.
According to spec, the JPEG_MARKER_DRI(Restart interval definition)
marker can come before the SOS marker. So we should not check the SOS
state when parsing the DRI marker.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5053>
-rw-r--r-- | subprojects/gst-plugins-bad/sys/va/gstjpegdecoder.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/subprojects/gst-plugins-bad/sys/va/gstjpegdecoder.c b/subprojects/gst-plugins-bad/sys/va/gstjpegdecoder.c index ce2c67dba1..2448c3cb5d 100644 --- a/subprojects/gst-plugins-bad/sys/va/gstjpegdecoder.c +++ b/subprojects/gst-plugins-bad/sys/va/gstjpegdecoder.c @@ -481,9 +481,10 @@ gst_jpeg_decoder_handle_frame (GstVideoDecoder * decoder, break; case GST_JPEG_MARKER_DRI: - if (!(valid_state (priv->state, GST_JPEG_DECODER_STATE_GOT_SOS) - && decode_restart_interval (self, &seg))) + if (!decode_restart_interval (self, &seg)) { + GST_WARNING_OBJECT (self, "Fail to decode restart interval"); goto unmap_and_error; + } break; case GST_JPEG_MARKER_DNL: break; |