GNOME Bugzilla – Bug 692265
codecparsers: h264: add inferred value for slice_beta_offset_div2
Last modified: 2013-06-12 09:49:29 UTC
Created attachment 234083 [details] [review] codecparsers: h264: add inferred value for slice_beta_offset_div2 Hi, the standard specifies that when slice_beta_offset_div2 is not present in the slice header, then the value of slice_beta_offset_div2 shall be inferred to be equal to 0. BTW, slice_alpha_c0_offset_div2 already had the same treatment. I plan to apply this patch to all GStreamer branches: 0.10, 1.0 and master.
commit 56e2405a3575a2326b496afc9901492c34450175 Author: Gwenole Beauchesne <gwenole.beauchesne@intel.com> Date: Sun Jan 6 18:54:52 2013 +0100 codecparsers: h264: add inferred value for slice_beta_offset_div2. The standard specifies that when slice_beta_offset_div2 is not present in the slice header, then the value of slice_beta_offset_div2 shall be inferred to be equal to 0. https://bugzilla.gnome.org/show_bug.cgi?id=692265 Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Hi, I am using gstreamer-1.0.4 version, which doesnt has this fix. What is the impact of this bug on the other elements like demuxers,decoders,encoders,muxerx?? regards JasonP
Hi, (In reply to comment #2) > I am using gstreamer-1.0.4 version, which doesnt has this fix. What is the > impact of this bug on the other elements like > demuxers,decoders,encoders,muxerx?? This bug only affected newer versions of gstreamer-vaapi, where we actually use that field. For other elements, there should not be any impact as this field was either not used, or the data structures were initially memset() to zero.
great. Thanks for you quick reply Gwenole. regards JasonP