annotate mjpegbdec.c @ 12530:63edd10ad4bc libavcodec tip

Try to fix crashes introduced by r25218 r25218 made assumptions about the existence of past reference frames that weren't necessarily true.
author darkshikari
date Tue, 28 Sep 2010 09:06:22 +0000
parents c35d7bc64882
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
1 /*
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
2 * Apple MJPEG-B decoder
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
3 * Copyright (c) 2002 Alex Beregszaszi
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
4 *
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
5 * This file is part of FFmpeg.
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
6 *
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
7 * FFmpeg is free software; you can redistribute it and/or
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
8 * modify it under the terms of the GNU Lesser General Public
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
9 * License as published by the Free Software Foundation; either
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
10 * version 2.1 of the License, or (at your option) any later version.
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
11 *
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
12 * FFmpeg is distributed in the hope that it will be useful,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
15 * Lesser General Public License for more details.
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
16 *
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
17 * You should have received a copy of the GNU Lesser General Public
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
18 * License along with FFmpeg; if not, write to the Free Software
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
20 */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
21
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
22 /**
11644
7dd2a45249a9 Remove explicit filename from Doxygen @file commands.
diego
parents: 11560
diff changeset
23 * @file
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
24 * Apple MJPEG-B decoder.
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
25 */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
26
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
27 #include "avcodec.h"
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
28 #include "mjpeg.h"
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
29 #include "mjpegdec.h"
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
30
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
31 static uint32_t read_offs(AVCodecContext *avctx, GetBitContext *gb, uint32_t size, const char *err_msg){
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
32 uint32_t offs= get_bits_long(gb, 32);
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
33 if(offs >= size){
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
34 av_log(avctx, AV_LOG_WARNING, err_msg, offs, size);
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
35 return 0;
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
36 }
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
37 return offs;
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
38 }
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
39
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
40 static int mjpegb_decode_frame(AVCodecContext *avctx,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
41 void *data, int *data_size,
9355
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8718
diff changeset
42 AVPacket *avpkt)
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
43 {
9355
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8718
diff changeset
44 const uint8_t *buf = avpkt->data;
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8718
diff changeset
45 int buf_size = avpkt->size;
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
46 MJpegDecodeContext *s = avctx->priv_data;
6271
michael
parents: 5985
diff changeset
47 const uint8_t *buf_end, *buf_ptr;
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
48 AVFrame *picture = data;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
49 GetBitContext hgb; /* for the header */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
50 uint32_t dqt_offs, dht_offs, sof_offs, sos_offs, second_field_offs;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
51 uint32_t field_size, sod_offs;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
52
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
53 buf_ptr = buf;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
54 buf_end = buf + buf_size;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
55
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
56 read_header:
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
57 /* reset on every SOI */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
58 s->restart_interval = 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
59 s->restart_count = 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
60 s->mjpb_skiptosod = 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
61
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
62 init_get_bits(&hgb, buf_ptr, /*buf_size*/(buf_end - buf_ptr)*8);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
63
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
64 skip_bits(&hgb, 32); /* reserved zeros */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
65
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
66 if (get_bits_long(&hgb, 32) != MKBETAG('m','j','p','g'))
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
67 {
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
68 av_log(avctx, AV_LOG_WARNING, "not mjpeg-b (bad fourcc)\n");
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
69 return 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
70 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
71
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
72 field_size = get_bits_long(&hgb, 32); /* field size */
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
73 av_log(avctx, AV_LOG_DEBUG, "field size: 0x%x\n", field_size);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
74 skip_bits(&hgb, 32); /* padded field size */
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
75 second_field_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "second_field_offs is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
76 av_log(avctx, AV_LOG_DEBUG, "second field offs: 0x%x\n", second_field_offs);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
77
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
78 dqt_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "dqt is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
79 av_log(avctx, AV_LOG_DEBUG, "dqt offs: 0x%x\n", dqt_offs);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
80 if (dqt_offs)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
81 {
5436
b4d41cc053b8 use buf_ptr to correctly decode second field
bcoudurier
parents: 5044
diff changeset
82 init_get_bits(&s->gb, buf_ptr+dqt_offs, (buf_end - (buf_ptr+dqt_offs))*8);
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
83 s->start_code = DQT;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
84 ff_mjpeg_decode_dqt(s);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
85 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
86
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
87 dht_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "dht is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
88 av_log(avctx, AV_LOG_DEBUG, "dht offs: 0x%x\n", dht_offs);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
89 if (dht_offs)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
90 {
5436
b4d41cc053b8 use buf_ptr to correctly decode second field
bcoudurier
parents: 5044
diff changeset
91 init_get_bits(&s->gb, buf_ptr+dht_offs, (buf_end - (buf_ptr+dht_offs))*8);
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
92 s->start_code = DHT;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
93 ff_mjpeg_decode_dht(s);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
94 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
95
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
96 sof_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "sof is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
97 av_log(avctx, AV_LOG_DEBUG, "sof offs: 0x%x\n", sof_offs);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
98 if (sof_offs)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
99 {
5436
b4d41cc053b8 use buf_ptr to correctly decode second field
bcoudurier
parents: 5044
diff changeset
100 init_get_bits(&s->gb, buf_ptr+sof_offs, (buf_end - (buf_ptr+sof_offs))*8);
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
101 s->start_code = SOF0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
102 if (ff_mjpeg_decode_sof(s) < 0)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
103 return -1;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
104 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
105
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
106 sos_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "sos is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
107 av_log(avctx, AV_LOG_DEBUG, "sos offs: 0x%x\n", sos_offs);
9737
f9769330c214 Make sure offsets of mjpeg b are within the buffer.
michael
parents: 9355
diff changeset
108 sod_offs = read_offs(avctx, &hgb, buf_end - buf_ptr, "sof is %d and size is %d\n");
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
109 av_log(avctx, AV_LOG_DEBUG, "sod offs: 0x%x\n", sod_offs);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
110 if (sos_offs)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
111 {
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
112 // init_get_bits(&s->gb, buf+sos_offs, (buf_end - (buf+sos_offs))*8);
5436
b4d41cc053b8 use buf_ptr to correctly decode second field
bcoudurier
parents: 5044
diff changeset
113 init_get_bits(&s->gb, buf_ptr+sos_offs, field_size*8);
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
114 s->mjpb_skiptosod = (sod_offs - sos_offs - show_bits(&s->gb, 16));
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
115 s->start_code = SOS;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
116 ff_mjpeg_decode_sos(s);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
117 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
118
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
119 if (s->interlaced) {
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
120 s->bottom_field ^= 1;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
121 /* if not bottom field, do not output image yet */
5985
51b4b51fa622 fix decoding of the first frame of gray.mov
michael
parents: 5436
diff changeset
122 if (s->bottom_field != s->interlace_polarity && second_field_offs)
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
123 {
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
124 buf_ptr = buf + second_field_offs;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
125 second_field_offs = 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
126 goto read_header;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
127 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
128 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
129
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
130 //XXX FIXME factorize, this looks very similar to the EOI code
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
131
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
132 *picture= s->picture;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
133 *data_size = sizeof(AVFrame);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
134
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
135 if(!s->lossless){
6655
22cca5d3173a Implement FFMAX3(a,b,c) - maximum over three arguments.
voroshil
parents: 6271
diff changeset
136 picture->quality= FFMAX3(s->qscale[0], s->qscale[1], s->qscale[2]);
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
137 picture->qstride= 0;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
138 picture->qscale_table= s->qscale_table;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
139 memset(picture->qscale_table, picture->quality, (s->width+15)/16);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
140 if(avctx->debug & FF_DEBUG_QP)
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
141 av_log(avctx, AV_LOG_DEBUG, "QP: %d\n", picture->quality);
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
142 picture->quality*= FF_QP2LAMBDA;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
143 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
144
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
145 return buf_ptr - buf;
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
146 }
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
147
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
148 AVCodec mjpegb_decoder = {
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
149 "mjpegb",
11560
8a4984c5cacc Define AVMediaType enum, and use it instead of enum CodecType, which
stefano
parents: 9737
diff changeset
150 AVMEDIA_TYPE_VIDEO,
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
151 CODEC_ID_MJPEGB,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
152 sizeof(MJpegDecodeContext),
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
153 ff_mjpeg_decode_init,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
154 NULL,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
155 ff_mjpeg_decode_end,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
156 mjpegb_decode_frame,
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
157 CODEC_CAP_DR1,
6717
5df0c730234d Add some long names to AVCodec declarations.
diego
parents: 6655
diff changeset
158 NULL,
12108
c35d7bc64882 Add new decoder property max_lowres and do not init decoder if requested value is higher.
cehoyos
parents: 11644
diff changeset
159 .max_lowres = 3,
7040
e943e1409077 Make AVCodec long_names definition conditional depending on CONFIG_SMALL.
stefano
parents: 6717
diff changeset
160 .long_name = NULL_IF_CONFIG_SMALL("Apple MJPEG-B"),
5044
924fdd6175b1 move mjpeg-b decoder in its own file
aurel
parents:
diff changeset
161 };