diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2021-08-29 22:18:52 +0200 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-09-19 17:05:33 +0200 |
| commit | e6bfb14223e44b4110c1d171d1ecffe9c80302a8 (patch) | |
| tree | 068cd711ecbc3523059a10c611b8a4923a78795c | |
| parent | c8707c105fc7cf3cd2ca59a5f2e645d129369d98 (diff) | |
| download | ffmpeg-e6bfb14223e44b4110c1d171d1ecffe9c80302a8.tar.gz | |
avcodec/aptxdec: Process data in complete blocks only
The APTX (HD) decoder decodes blocks of four (six) bytes to four
output samples. It makes no sense to handle incomplete blocks:
They would just lead to synchronization errors, in which case
the complete frame is discarded. So only handle complete blocks.
This also avoids reading from the packet's padding and writing
into the frame's padding.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
| -rw-r--r-- | libavcodec/aptxdec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/aptxdec.c b/libavcodec/aptxdec.c index 878c9ffe1b..d254b3026b 100644 --- a/libavcodec/aptxdec.c +++ b/libavcodec/aptxdec.c @@ -151,7 +151,7 @@ static int aptx_decode_frame(AVCodecContext *avctx, AVFrame *frame, /* get output buffer */ frame->ch_layout.nb_channels = NB_CHANNELS; frame->format = AV_SAMPLE_FMT_S32P; - frame->nb_samples = 4 * avpkt->size / s->block_size; + frame->nb_samples = 4 * (avpkt->size / s->block_size); if ((ret = ff_get_buffer(avctx, frame, 0)) < 0) return ret; |