codecparsers: h264: fix picture level scaling lists derivation (rule B).
authorCong Zhong <congx.zhong@intel.com>
Wed, 17 Apr 2013 02:44:48 +0000 (10:44 +0800)
committerTim-Philipp Müller <tim@centricular.com>
Thu, 12 Dec 2013 23:47:29 +0000 (23:47 +0000)
Fix picture level scaling lists derivation from fall-back rule set B,
as specified in 7.4.2.2. More precisely, the sequence level scaling
lists need to be used but intra and inter lists arguments were swapped.

This fixes FRExt/freh5.264 from conformance testing.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
gst-libs/gst/codecparsers/gsth264parser.c

index 3173205c9959c4e04ce24b14d6896733a0ab7527..bd79a6fafc6872762aa113e9f0b784e85c830cc6 100644 (file)
@@ -1721,8 +1721,8 @@ gst_h264_parse_pps (GstH264NalParser * nalparser, GstH264NalUnit * nalu,
     if (sps->scaling_matrix_present_flag) {
       if (!gst_h264_parser_parse_scaling_list (&nr,
               pps->scaling_lists_4x4, pps->scaling_lists_8x8,
-              sps->scaling_lists_4x4[0], sps->scaling_lists_4x4[3],
-              sps->scaling_lists_8x8[0], sps->scaling_lists_8x8[3], n_lists))
+              sps->scaling_lists_4x4[3], sps->scaling_lists_4x4[0],
+              sps->scaling_lists_8x8[3], sps->scaling_lists_8x8[0], n_lists))
         goto error;
     } else {
       if (!gst_h264_parser_parse_scaling_list (&nr,