Re: [FFmpeg-devel] [PATCH] sws_scale: Use av_sat_add32 in yuv2rgba64 template

[Michael Niedermayer <michael@niedermayer.cc>]

[-- Attachment #1.1: Type: text/plain, Size: 1294 bytes --]

dooa    2On Tue, Nov 01, 2022 at 03:12:08PM -0400, Drew Dunne wrote:
> This was supposed to be in reply to a previous patch I sent, but I changed
> the git message so maybe it didn't like that. Sorry about that.
> 
> Attached is the example YUVA file that can reproduce the overflow bug.
> 
> The command to reproduce is:
> 
> ./ffmpeg \
>   -f rawvideo -video_size 66x64 -pixel_format yuva420p10le \
>   -i overflow_input_w66h64.yuva420p10le \
>   -filter_complex
> "scale=flags=bicubic+full_chroma_int+full_chroma_inp+bitexact+accurate_rnd:in_color_matrix=bt2020:out_color_matrix=bt2020:in_range=full:out_range=full,format=rgba64[out]"
> \
>   -f rawvideo -codec:v:0 rawvideo -pixel_format rgba64 -map '[out]' \
>   -y overflow_w66h64.rgba64
> 
> I've attached a PNG of the resulting overflowed image, there is a clear
> discoloration in the bottom left corner that's orange instead of pink.
> 
> I've also attached a PNG of the image using the patch to correct this
> overflow.

I dont like the multi cliping per sample, ill post a different solution

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Elect your leaders based on what they did after the last election, not
based on what they say before an election.


[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".