aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2023-06-08 10:26:35 -0400
committerLeo Izen <leo.izen@gmail.com>2023-06-11 21:11:39 -0400
commit09621fd7d93a12974e9664b2aebb8237e5c46f03 (patch)
treefa1c38058d0f3f61d6c57ba19b7a2930fb0b2d50
parent1ec4553e355039ce69abf8e49389fa43f1f55fc5 (diff)
downloadffmpeg-09621fd7d93a12974e9664b2aebb8237e5c46f03.tar.gz
avformat/jpegxl_probe: Forward error codes
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavformat/jpegxl_probe.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/libavformat/jpegxl_probe.c b/libavformat/jpegxl_probe.c
index e15e9eee49..88492cb772 100644
--- a/libavformat/jpegxl_probe.c
+++ b/libavformat/jpegxl_probe.c
@@ -261,8 +261,8 @@ int ff_jpegxl_verify_codestream_header(const uint8_t *buf, int buflen, int valid
if (get_bits_long(gb, 16) != FF_JPEGXL_CODESTREAM_SIGNATURE_LE)
return -1;
- if (jpegxl_read_size_header(gb) < 0 && validate_level)
- return -1;
+ if ((ret = jpegxl_read_size_header(gb)) < 0 && validate_level)
+ return ret;
all_default = get_bits1(gb);
if (!all_default)
@@ -281,8 +281,9 @@ int ff_jpegxl_verify_codestream_header(const uint8_t *buf, int buflen, int valid
/* preview header */
if (get_bits1(gb)) {
- if (jpegxl_read_preview_header(gb) < 0)
- return -1;
+ ret = jpegxl_read_preview_header(gb);
+ if (ret < 0)
+ return ret;
}
/* animation header */
@@ -308,8 +309,9 @@ int ff_jpegxl_verify_codestream_header(const uint8_t *buf, int buflen, int valid
if (num_extra_channels > 4 && validate_level)
return -1;
for (uint32_t i = 0; i < num_extra_channels; i++) {
- if (jpegxl_read_extra_channel_info(gb, validate_level) < 0)
- return -1;
+ ret = jpegxl_read_extra_channel_info(gb, validate_level);
+ if (ret < 0)
+ return ret;
if (get_bits_left(gb) < 1)
return AVERROR_INVALIDDATA;
}