From a81af56db9ad0b723a231739fefcd009f018682b Mon Sep 17 00:00:00 2001 From: Vincent Rabaud Date: Wed, 23 Jul 2025 11:08:02 +0200 Subject: [PATCH] src/dec: fix a status reported during setup If setup() fails, it is not because the user did an abort according the doc : https://chromium.googlesource.com/webm/libwebp/+/refs/tags/v1.6.0/src/dec/vp8_dec.h#74 Bug: webp:433456394 Change-Id: I25020304ccdb8eae32134c1400a918e7c4cc0385 --- src/dec/frame_dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/dec/frame_dec.c b/src/dec/frame_dec.c index 87807722..8042f2c2 100644 --- a/src/dec/frame_dec.c +++ b/src/dec/frame_dec.c @@ -561,7 +561,7 @@ VP8StatusCode VP8EnterCritical(VP8Decoder* const dec, VP8Io* const io) { // Call setup() first. This may trigger additional decoding features on 'io'. // Note: Afterward, we must call teardown() no matter what. if (io->setup != NULL && !io->setup(io)) { - VP8SetError(dec, VP8_STATUS_USER_ABORT, "Frame setup failed"); + VP8SetError(dec, VP8_STATUS_INVALID_PARAM, "Frame setup failed"); return dec->status; }