[FFmpeg-devel] [PATCH 3/3] avformat/v210: Check width and height
Timothy Gu
timothygu99 at gmail.com
Sat Nov 21 06:34:13 CET 2015
Fixes a floating point exception when width and height are not supplied
(and therefore are zero).
---
libavformat/v210.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/libavformat/v210.c b/libavformat/v210.c
index ab29171..7503d5d 100644
--- a/libavformat/v210.c
+++ b/libavformat/v210.c
@@ -39,6 +39,7 @@ static int v210_read_header(AVFormatContext *ctx)
{
V210DemuxerContext *s = ctx->priv_data;
AVStream *st;
+ int ret;
st = avformat_new_stream(ctx, NULL);
if (!st)
@@ -50,11 +51,15 @@ static int v210_read_header(AVFormatContext *ctx)
avpriv_set_pts_info(st, 64, s->framerate.den, s->framerate.num);
+ ret = av_image_check_size(s->width, s->height, 0, ctx);
+ if (ret < 0)
+ return ret;
st->codec->width = s->width;
st->codec->height = s->height;
st->codec->pix_fmt = ctx->iformat->raw_codec_id == AV_CODEC_ID_V210 ?
AV_PIX_FMT_YUV422P10 : AV_PIX_FMT_YUV422P16;
- st->codec->bit_rate = av_rescale_q(GET_PACKET_SIZE(s->width, s->height),
+ ctx->packet_size = GET_PACKET_SIZE(s->width, s->height);
+ st->codec->bit_rate = av_rescale_q(ctx->packet_size,
(AVRational){8,1}, st->time_base);
return 0;
@@ -63,18 +68,14 @@ static int v210_read_header(AVFormatContext *ctx)
static int v210_read_packet(AVFormatContext *s, AVPacket *pkt)
{
- int packet_size, ret, width, height;
+ int ret, width, height;
AVStream *st = s->streams[0];
width = st->codec->width;
height = st->codec->height;
- packet_size = GET_PACKET_SIZE(width, height);
- if (packet_size < 0)
- return -1;
-
- ret = av_get_packet(s->pb, pkt, packet_size);
- pkt->pts = pkt->dts = pkt->pos / packet_size;
+ ret = av_get_packet(s->pb, pkt, s->packet_size);
+ pkt->pts = pkt->dts = pkt->pos / s->packet_size;
pkt->stream_index = 0;
if (ret < 0)
--
2.1.4
More information about the ffmpeg-devel
mailing list