Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_testsrc: fix colorchart black stripe

[Michael Niedermayer <michael@niedermayer.cc>]

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

On Fri, Jan 12, 2024 at 01:58:03PM +0200, Vladimir Petrov wrote:
> There is a typo at 'libavfilter/vsrc_testsrc.c' in
> 'colorchart_fill_picture()' regarding patch height when using
> colorchart test image source. With current default size 64x64 all is
> fine, but when move to any non-square patch size, there is unexpected
> behavior:
> 
> Test examples:
> 
> [Blackstripe on bottom if height > width]
> ffmpeg -f lavfi -i colorchart=duration=30:rate=25:patch_size=120x144
> -f mp4 -c:v libx264 colorchart1.m4v
> 
> [Segmentation fault if height < width]
> ffmpeg -f lavfi -i colorchart=duration=30:rate=25:patch_size=120x100
> -f mp4 -c:v libx264 colorchart2.m4v

>  vsrc_testsrc.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 0a5e124445e68daed7d6481a25cc877d45696b8e  0001-avfilter-vsrc_testsrc-fix-colorchart-black-stripe.patch
> From c484a4976c54083770c8749f954394c35dfd9c63 Mon Sep 17 00:00:00 2001
> From: Vladimir Petrov <vppetrovmms@gmail.com>
> Date: Tue, 9 Jan 2024 18:42:19 +0200
> Subject: [PATCH] avfilter/vsrc_testsrc: fix colorchart black stripe
> 
> Fixed blackstripe on bottom or segmentation fault in case
> when patch width and height differ.
> 
> Signed-off-by: Vladimir Petrov <vppetrovmms@gmail.com>
> ---
>  libavfilter/vsrc_testsrc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

will apply

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Take away the freedom of one citizen and you will be jailed, take away
the freedom of all citizens and you will be congratulated by your peers
in Parliament.

[-- 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".