[FFmpeg-devel] [PATCH 1/3] - libavcodec/aom_film_grain: Handle byte alignment when multiple film grain parameters sets are present in the bitstream

[Michael Niedermayer]

On Fri, Sep 20, 2024 at 12:42:47AM +0000, Segall, Andrew via ffmpeg-devel wrote:
> More info: Series of patches to align the implementation of aom_film_grain.c with the AOM specification.
> 
> First time committing.  Suggestions very welcome.
> 
> Signed-off-by: Andrew Segall <asegall at amazon.com <mailto:asegall at amazon.com>>
> ---
> libavcodec/aom_film_grain.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
> 
> 
> diff --git a/libavcodec/aom_film_grain.c b/libavcodec/aom_film_grain.c
> index e302567ba5..fdfa3dbf77 100644
> --- a/libavcodec/aom_film_grain.c
> +++ b/libavcodec/aom_film_grain.c
> @@ -151,12 +151,24 @@ int ff_aom_parse_film_grain_sets(AVFilmGrainAFGS1Params *s,
> 
> 
> fgp->type = get_bits1(gb) ? AV_FILM_GRAIN_PARAMS_AV1 : AV_FILM_GRAIN_PARAMS_NONE;
> if (!fgp->type)
> + {
> + payload_bits = get_bits_count(gb) - start_position;
> + if (payload_bits > payload_size * 8)
> + goto error;
> + skip_bits(gb, payload_size * 8 - payload_bits);
> continue;
> + }

patch is corrupted by linebreaks / wordwrap, check youe MUA/editor settings

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If the United States is serious about tackling the national security threats 
related to an insecure 5G network, it needs to rethink the extent to which it
values corporate profits and government espionage over security.-Bruce Schneier
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240920/4f14ef50/attachment.sig>