diff options
| author | Paul B Mahol <onemda@gmail.com> | 2015-06-06 15:56:06 +0000 |
|---|---|---|
| committer | Paul B Mahol <onemda@gmail.com> | 2015-06-06 16:02:19 +0000 |
| commit | a03b69478b7f1c0c31e53acb0cf392917c0f967a (patch) | |
| tree | eb8e8194007b9bb08151ecc7f32d61409f095f36 /libavcodec/exr.c | |
| parent | 1b4dd59e5fbdebb8d9f13ad2dbdaa0179d0cce57 (diff) | |
| download | ffmpeg-a03b69478b7f1c0c31e53acb0cf392917c0f967a.tar.gz | |
avcodec/exr: fix crash caused by merge
Various header informations need to be reset when decoding next frame.
Regression since: 95582b5c
Fixes ticket #4597.
Signed-off-by: Paul B Mahol <onemda@gmail.com>
Diffstat (limited to 'libavcodec/exr.c')
| -rw-r--r-- | libavcodec/exr.c | 31 |
1 files changed, 16 insertions, 15 deletions
diff --git a/libavcodec/exr.c b/libavcodec/exr.c index f9525ecf02..9308ae03a8 100644 --- a/libavcodec/exr.c +++ b/libavcodec/exr.c @@ -1010,6 +1010,22 @@ static int decode_header(EXRContext *s) int current_channel_offset = 0; int magic_number, version, flags, i; + s->xmin = ~0; + s->xmax = ~0; + s->ymin = ~0; + s->ymax = ~0; + s->xdelta = ~0; + s->ydelta = ~0; + s->channel_offsets[0] = -1; + s->channel_offsets[1] = -1; + s->channel_offsets[2] = -1; + s->channel_offsets[3] = -1; + s->pixel_type = EXR_UNKNOWN; + s->compression = EXR_UNKN; + s->nb_channels = 0; + s->w = 0; + s->h = 0; + if (bytestream2_get_bytes_left(&s->gb) < 10) { av_log(s->avctx, AV_LOG_ERROR, "Header too short to parse.\n"); return AVERROR_INVALIDDATA; @@ -1350,21 +1366,6 @@ static av_cold int decode_init(AVCodecContext *avctx) float one_gamma = 1.0f / s->gamma; s->avctx = avctx; - s->xmin = ~0; - s->xmax = ~0; - s->ymin = ~0; - s->ymax = ~0; - s->xdelta = ~0; - s->ydelta = ~0; - s->channel_offsets[0] = -1; - s->channel_offsets[1] = -1; - s->channel_offsets[2] = -1; - s->channel_offsets[3] = -1; - s->pixel_type = EXR_UNKNOWN; - s->compression = EXR_UNKN; - s->nb_channels = 0; - s->w = 0; - s->h = 0; if (one_gamma > 0.9999f && one_gamma < 1.0001f) { for (i = 0; i < 65536; ++i) |