From e876f605c834766ce312372f55271e97e8327a8b Mon Sep 17 00:00:00 2001 From: Mengkejiergeli Ba Date: Mon, 10 Jan 2022 15:24:13 +0800 Subject: av1parser: Fix data type of film grain param Fix cb_offset and cr_offset data type from guint8 to guint16. According to spec, cb_offset and cr_offset are 9 bit long, while guint8 can cause interger overflow, and thus change to guint16. Part-of: --- gst-libs/gst/codecparsers/gstav1parser.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gst-libs/gst/codecparsers/gstav1parser.h b/gst-libs/gst/codecparsers/gstav1parser.h index bdbe3e11a..4b49a356e 100644 --- a/gst-libs/gst/codecparsers/gstav1parser.h +++ b/gst-libs/gst/codecparsers/gstav1parser.h @@ -1370,10 +1370,10 @@ struct _GstAV1FilmGrainParams { guint8 grain_scale_shift; guint8 cb_mult; guint8 cb_luma_mult; - guint8 cb_offset; + guint16 cb_offset; guint8 cr_mult; guint8 cr_luma_mult; - guint8 cr_offset; + guint16 cr_offset; gboolean overlap_flag; gboolean clip_to_restricted_range; }; -- cgit v1.2.1