[Bug 721384] h264parse: clears keyframe flags when passing through byte-stream AUs (regression)

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=721384
  GStreamer | gst-plugins-bad | git

--- Comment #7 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2014-01-20 20:34:59 UTC ---
Created an attachment (id=266802)
 View: https://bugzilla.gnome.org/attachment.cgi?id=266802
 Review: https://bugzilla.gnome.org/review?bug=721384&attachment=266802

h264parser: be forgiving when reading second_chroma_qp_index_offset

This makes the file play again, but it is a hack.

I see 2 options here:

1) this hack (that is very specific to this particular error) to
   ignore errors when reading the second_chroma_qp_index_offset, which
   is the last of the fields on the PPS

2) Make the UE/SE read accept 0 as 0 and valid, making a single bit 0 mean
   the same as a single bit 1. This has a broader range, but also can lead
   to more side effects

I don't know h264 spec enough to decide :)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.