You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-15 14:13:16 +02:00
Merge commit '8aba7968dd604aae91ee42cbce0be3dad7dceb30'
* commit '8aba7968dd604aae91ee42cbce0be3dad7dceb30': vcr1: add sanity checks Conflicts: libavcodec/vcr1.c See:845724c82c
See:8e09482e4d
Merged-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
@@ -38,9 +38,10 @@ static av_cold int vcr1_decode_init(AVCodecContext *avctx)
|
|||||||
avctx->pix_fmt = AV_PIX_FMT_YUV410P;
|
avctx->pix_fmt = AV_PIX_FMT_YUV410P;
|
||||||
|
|
||||||
if (avctx->width % 8 || avctx->height%4) {
|
if (avctx->width % 8 || avctx->height%4) {
|
||||||
avpriv_request_sample(avctx, "odd dimensions support");
|
avpriv_request_sample(avctx, "odd dimensions (%d x %d) support", avctx->width, avctx->height);
|
||||||
return AVERROR_PATCHWELCOME;
|
return AVERROR_INVALIDDATA;
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -64,9 +65,13 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
|
|||||||
p->pict_type = AV_PICTURE_TYPE_I;
|
p->pict_type = AV_PICTURE_TYPE_I;
|
||||||
p->key_frame = 1;
|
p->key_frame = 1;
|
||||||
|
|
||||||
|
if (buf_size < 32)
|
||||||
|
goto packet_small;
|
||||||
|
|
||||||
for (i = 0; i < 16; i++) {
|
for (i = 0; i < 16; i++) {
|
||||||
a->delta[i] = *bytestream++;
|
a->delta[i] = *bytestream++;
|
||||||
bytestream++;
|
bytestream++;
|
||||||
|
buf_size--;
|
||||||
}
|
}
|
||||||
|
|
||||||
for (y = 0; y < avctx->height; y++) {
|
for (y = 0; y < avctx->height; y++) {
|
||||||
@@ -77,8 +82,12 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
|
|||||||
uint8_t *cb = &p->data[1][(y >> 2) * p->linesize[1]];
|
uint8_t *cb = &p->data[1][(y >> 2) * p->linesize[1]];
|
||||||
uint8_t *cr = &p->data[2][(y >> 2) * p->linesize[2]];
|
uint8_t *cr = &p->data[2][(y >> 2) * p->linesize[2]];
|
||||||
|
|
||||||
|
if (buf_size < 4 + avctx->width)
|
||||||
|
goto packet_small;
|
||||||
|
|
||||||
for (i = 0; i < 4; i++)
|
for (i = 0; i < 4; i++)
|
||||||
a->offset[i] = *bytestream++;
|
a->offset[i] = *bytestream++;
|
||||||
|
buf_size -= 4;
|
||||||
|
|
||||||
offset = a->offset[0] - a->delta[bytestream[2] & 0xF];
|
offset = a->offset[0] - a->delta[bytestream[2] & 0xF];
|
||||||
for (x = 0; x < avctx->width; x += 4) {
|
for (x = 0; x < avctx->width; x += 4) {
|
||||||
@@ -92,8 +101,12 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
|
|||||||
*cr++ = bytestream[1];
|
*cr++ = bytestream[1];
|
||||||
|
|
||||||
bytestream += 4;
|
bytestream += 4;
|
||||||
|
buf_size -= 4;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
|
if (buf_size < avctx->width / 2)
|
||||||
|
goto packet_small;
|
||||||
|
|
||||||
offset = a->offset[y & 3] - a->delta[bytestream[2] & 0xF];
|
offset = a->offset[y & 3] - a->delta[bytestream[2] & 0xF];
|
||||||
|
|
||||||
for (x = 0; x < avctx->width; x += 8) {
|
for (x = 0; x < avctx->width; x += 8) {
|
||||||
@@ -107,6 +120,7 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
|
|||||||
luma[7] = offset += a->delta[bytestream[1] >> 4];
|
luma[7] = offset += a->delta[bytestream[1] >> 4];
|
||||||
luma += 8;
|
luma += 8;
|
||||||
bytestream += 4;
|
bytestream += 4;
|
||||||
|
buf_size -= 4;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -114,6 +128,9 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
|
|||||||
*got_frame = 1;
|
*got_frame = 1;
|
||||||
|
|
||||||
return buf_size;
|
return buf_size;
|
||||||
|
packet_small:
|
||||||
|
av_log(avctx, AV_LOG_ERROR, "Input packet too small.\n");
|
||||||
|
return AVERROR_INVALIDDATA;
|
||||||
}
|
}
|
||||||
|
|
||||||
AVCodec ff_vcr1_decoder = {
|
AVCodec ff_vcr1_decoder = {
|
||||||
|
Reference in New Issue
Block a user