Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* Re: [FFmpeg-devel] [PATCH] avcodec/webp: fix incorrectly returning AVERROR_INVALIDDATA when building Huffman reader (PR #20316)
       [not found] <20250822173828.8059368E6BF@ffbox0-bg.ffmpeg.org>
@ 2025-08-25 15:59 ` Raphael Zumer via ffmpeg-devel
  2025-08-26 14:47   ` [FFmpeg-devel] " Derek Buitenhuis via ffmpeg-devel
  0 siblings, 1 reply; 2+ messages in thread
From: Raphael Zumer via ffmpeg-devel @ 2025-08-25 15:59 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Raphael Zumer

Hello,

Looking for a review for this patch, as it currently breaks WebP decoding and is a 1-line fix.

Thanks,
RZ

On 8/22/25 13:38, rzumer via ffmpeg-devel wrote:
> PR #20316 opened by rzumer
> URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20316
> Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20316.patch
>
> This corrects an error in commit c33f16d, causing the single-symbol special case to fall through to the no-symbol one and returning AVERROR_INVALIDDATA instead of a success status code. Since the same commit removes shared logic between the two special cases, I split them up to simplify it as a follow-up.
>
>
> From c031ae60335ed0f02b5b8c80a069656b68f66b68 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Rapha=C3=ABl=20Zumer?= <rzumer@tebako.net>
> Date: Fri, 22 Aug 2025 12:44:09 -0400
> Subject: [PATCH 1/2] avcodec/webp: fix incorrectly returning
>  AVERROR_INVALIDDATA when building Huffman reader
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> This corrects an error in commit c33f16d.
>
> Signed-off-by: Raphaël Zumer <rzumer@tebako.net>
> ---
>  libavcodec/webp.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavcodec/webp.c b/libavcodec/webp.c
> index 56c3ec6d28..a350dff7a7 100644
> --- a/libavcodec/webp.c
> +++ b/libavcodec/webp.c
> @@ -281,6 +281,7 @@ static int huff_reader_build_canonical(HuffReader *r, const uint8_t *code_length
>              r->nb_symbols = 1;
>              r->simple = 1;
>              r->simple_symbols[0] = syms[0];
> +            return 0;
>          }
>          // No symbols
>          return AVERROR_INVALIDDATA;
_______________________________________________
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".

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [FFmpeg-devel] Re: [PATCH] avcodec/webp: fix incorrectly returning AVERROR_INVALIDDATA when building Huffman reader (PR #20316)
  2025-08-25 15:59 ` [FFmpeg-devel] [PATCH] avcodec/webp: fix incorrectly returning AVERROR_INVALIDDATA when building Huffman reader (PR #20316) Raphael Zumer via ffmpeg-devel
@ 2025-08-26 14:47   ` Derek Buitenhuis via ffmpeg-devel
  0 siblings, 0 replies; 2+ messages in thread
From: Derek Buitenhuis via ffmpeg-devel @ 2025-08-26 14:47 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Derek Buitenhuis

On 8/25/2025 4:59 PM, Raphael Zumer via ffmpeg-devel wrote:
> Looking for a review for this patch, as it currently breaks WebP decoding and is a 1-line fix

Reviewed and merged.

This needs to be backported to 8.0 as it is a major regression in WebP decoding.

- Derek
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-08-26 14:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20250822173828.8059368E6BF@ffbox0-bg.ffmpeg.org>
2025-08-25 15:59 ` [FFmpeg-devel] [PATCH] avcodec/webp: fix incorrectly returning AVERROR_INVALIDDATA when building Huffman reader (PR #20316) Raphael Zumer via ffmpeg-devel
2025-08-26 14:47   ` [FFmpeg-devel] " Derek Buitenhuis via ffmpeg-devel

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git