diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2024-04-28 22:57:53 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2024-05-01 02:57:11 +0200 |
| commit | 615c994739cacbeb0a2f48f8271d911fcd0b4303 (patch) | |
| tree | f84e1a05b4b2b9c397bac7674d098fa1fb6b3efe | |
| parent | 63415168dbd96475372e37ae0fd47bafe151e2f0 (diff) | |
| download | ffmpeg-615c994739cacbeb0a2f48f8271d911fcd0b4303.tar.gz | |
avcodec/atrac9dec: Check init_get_bits8() for failure
Fixes: CID1439569 Unchecked return value
Fixes: CID1439578 Unchecked return value
Sponsored-by: Sovereign Tech Fund
Reviewed-by: Lynne <dev@lynne.ee>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/atrac9dec.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/libavcodec/atrac9dec.c b/libavcodec/atrac9dec.c index df68407af9..e375f46fd0 100644 --- a/libavcodec/atrac9dec.c +++ b/libavcodec/atrac9dec.c @@ -802,7 +802,9 @@ static int atrac9_decode_frame(AVCodecContext *avctx, AVFrame *frame, if (ret < 0) return ret; - init_get_bits8(&gb, avpkt->data, avpkt->size); + ret = init_get_bits8(&gb, avpkt->data, avpkt->size); + if (ret < 0) + return ret; for (int i = 0; i < frames; i++) { for (int j = 0; j < s->block_config->count; j++) { @@ -922,7 +924,9 @@ static av_cold int atrac9_decode_init(AVCodecContext *avctx) return AVERROR_INVALIDDATA; } - init_get_bits8(&gb, avctx->extradata + 4, avctx->extradata_size); + err = init_get_bits8(&gb, avctx->extradata + 4, avctx->extradata_size); + if (err < 0) + return err; if (get_bits(&gb, 8) != 0xFE) { av_log(avctx, AV_LOG_ERROR, "Incorrect magic byte!\n"); |