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
This commit is contained in:
Vincent Rabaud
2025-07-23 11:08:02 +02:00
parent 4e63ff1eb6
commit a81af56db9

View File

@@ -561,7 +561,7 @@ VP8StatusCode VP8EnterCritical(VP8Decoder* const dec, VP8Io* const io) {
// Call setup() first. This may trigger additional decoding features on 'io'. // Call setup() first. This may trigger additional decoding features on 'io'.
// Note: Afterward, we must call teardown() no matter what. // Note: Afterward, we must call teardown() no matter what.
if (io->setup != NULL && !io->setup(io)) { 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; return dec->status;
} }