[Bug 786797] codecparsers: h264: store SPS's and subset SPS's separately

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=786797

--- Comment #1 from sreerenj <bsreerenj at gmail.com> ---

It seems there are samples with the same value of sequence_parameter_set_id in
SPS and SSPS. Current h264 codecparser implementation using the same array for
storing sps and ssps,  then sps get overridden by ssps if both have the same
id. As per the spec "SPS should be replaced by another SPS of same
sequence_parameter_set_id" AND "SSPS should be replaced by another SSPS of same
sequece_parameter_set_id".  

I think we should use a separate array "GstH264SPS ssps[]" and "GstH264SPS
*last_ssps" in H264NalParser.
And changes required in many places for MVC parsing (SVC isn't implemented
yet).

It is going to bring an ABI breakage too.

@Orestis: Can you please attach the sample stream which has this issue?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.