GNOME Bugzilla – Bug 768532
h265parse: do not forget VPS/SPS/PPS after pushing codec header
Last modified: 2018-11-03 13:53:13 UTC
Created attachment 331030 [details] [review] 0001-h265parse-do-not-forget-VPS-SPS-PPS-after-pushing-co.patch have_vps, have_sps and have_pps are used to store if VPS/SPS/PPS headers are found. This flags should not be cleared if the codec header is pushed, only when the stream is reset. In h264parse, this variables are used to store if *new* codec headers are found and a flag is used to indicate the presence of the codec headers. This fixes streaming with config-interval>0.
The use of have_vps, have_sps and have_pps seems a bit strange even before this change. h265parse->have_vps is written but never read, for example. And h265parse->push_codec seems to be set to FALSE if VPS and PPS have been processed in gst_h265_parse_process_nal() (it doesn't seem to care about SPS in that case).
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/408.