annotate xsubdec.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 ffb3668ff7af
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
5493
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
1 /*
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
2 * XSUB subtitle decoder
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
3 * Copyright (c) 2007 Reimar Döffinger
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
4 *
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
5 * This file is part of FFmpeg.
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
6 *
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
7 * FFmpeg is free software; you can redistribute it and/or
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
8 * modify it under the terms of the GNU Lesser General Public
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
9 * License as published by the Free Software Foundation; either
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
10 * version 2.1 of the License, or (at your option) any later version.
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
11 *
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
12 * FFmpeg is distributed in the hope that it will be useful,
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
15 * Lesser General Public License for more details.
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
16 *
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
17 * You should have received a copy of the GNU Lesser General Public
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
18 * License along with FFmpeg; if not, write to the Free Software
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
80b481ec72a6 Add proper license header
reimar
parents: 5492
diff changeset
20 */
12372
914f484bb476 Remove use of the deprecated function avcodec_check_dimensions(), use
stefano
parents: 11560
diff changeset
21 #include "libavcore/imgutils.h"
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
22 #include "avcodec.h"
9428
0dce4fe6e6f3 Rename bitstream.h to get_bits.h.
stefano
parents: 9355
diff changeset
23 #include "get_bits.h"
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
24 #include "bytestream.h"
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
25
6517
48759bfbd073 Apply 'cold' attribute to init/uninit functions in libavcodec
zuxy
parents: 6218
diff changeset
26 static av_cold int decode_init(AVCodecContext *avctx) {
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
27 avctx->pix_fmt = PIX_FMT_PAL8;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
28 return 0;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
29 }
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
30
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
31 static const uint8_t tc_offsets[9] = { 0, 1, 3, 4, 6, 7, 9, 10, 11 };
8769
6fc3497dc81f Fix timestamp parsing in XSUB decoder.
cehoyos
parents: 8516
diff changeset
32 static const uint8_t tc_muls[9] = { 10, 6, 10, 6, 10, 10, 10, 10, 1 };
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
33
10054
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
34 static int64_t parse_timecode(const uint8_t *buf, int64_t packet_time) {
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
35 int i;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
36 int64_t ms = 0;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
37 if (buf[2] != ':' || buf[5] != ':' || buf[8] != '.')
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
38 return AV_NOPTS_VALUE;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
39 for (i = 0; i < sizeof(tc_offsets); i++) {
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
40 uint8_t c = buf[tc_offsets[i]] - '0';
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
41 if (c > 9) return AV_NOPTS_VALUE;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
42 ms = (ms + c) * tc_muls[i];
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
43 }
10054
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
44 return ms - packet_time;
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
45 }
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
46
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
47 static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
9355
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8810
diff changeset
48 AVPacket *avpkt) {
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8810
diff changeset
49 const uint8_t *buf = avpkt->data;
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 8810
diff changeset
50 int buf_size = avpkt->size;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
51 AVSubtitle *sub = data;
6218
michael
parents: 5982
diff changeset
52 const uint8_t *buf_end = buf + buf_size;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
53 uint8_t *bitmap;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
54 int w, h, x, y, rlelen, i;
10054
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
55 int64_t packet_time = 0;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
56 GetBitContext gb;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
57
10050
b8313eb3dca8 Make sure AVSubtitle is initialized, memset it to 0.
reimar
parents: 9988
diff changeset
58 memset(sub, 0, sizeof(*sub));
8810
367e4d405cd9 Set AVSubtitle format to 0. Neither dvdsubdec nor xsubdec intializes format
diego
parents: 8769
diff changeset
59
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
60 // check that at least header fits
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
61 if (buf_size < 27 + 7 * 2 + 4 * 3) {
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
62 av_log(avctx, AV_LOG_ERROR, "coded frame too small\n");
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
63 return -1;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
64 }
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
65
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
66 // read start and end time
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
67 if (buf[0] != '[' || buf[13] != '-' || buf[26] != ']') {
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
68 av_log(avctx, AV_LOG_ERROR, "invalid time code\n");
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
69 return -1;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
70 }
10054
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
71 if (avpkt->pts != AV_NOPTS_VALUE)
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
72 packet_time = av_rescale_q(avpkt->pts, AV_TIME_BASE_Q, (AVRational){1, 1000});
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
73 sub->start_display_time = parse_timecode(buf + 1, packet_time);
8ab9fbad11b2 Fix start_display_time/end_display_time to be relative to packet pts in xsub decoder.
reimar
parents: 10053
diff changeset
74 sub->end_display_time = parse_timecode(buf + 14, packet_time);
5484
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
75 buf += 27;
ad3e24850727 Add forgotten xsub timecode parsing
reimar
parents: 5483
diff changeset
76
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
77 // read header
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
78 w = bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
79 h = bytestream_get_le16(&buf);
12462
ffb3668ff7af Use new imgutils.h API names, fix deprecation warnings.
stefano
parents: 12372
diff changeset
80 if (av_image_check_size(w, h, 0, avctx) < 0)
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
81 return -1;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
82 x = bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
83 y = bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
84 // skip bottom right position, it gives no new information
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
85 bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
86 bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
87 rlelen = bytestream_get_le16(&buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
88
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
89 // allocate sub and set values
10053
089dd845c8c0 Reindent.
reimar
parents: 10052
diff changeset
90 sub->rects = av_mallocz(sizeof(*sub->rects));
089dd845c8c0 Reindent.
reimar
parents: 10052
diff changeset
91 sub->rects[0] = av_mallocz(sizeof(*sub->rects[0]));
089dd845c8c0 Reindent.
reimar
parents: 10052
diff changeset
92 sub->num_rects = 1;
8512
aa45029f5cd7 Change AVSubtitle.rects to an array of pointers so ABI does not break
michael
parents: 7040
diff changeset
93 sub->rects[0]->x = x; sub->rects[0]->y = y;
aa45029f5cd7 Change AVSubtitle.rects to an array of pointers so ABI does not break
michael
parents: 7040
diff changeset
94 sub->rects[0]->w = w; sub->rects[0]->h = h;
9988
b8a9cfe64488 Set subtitle type in DVD and XSUB subtitle decoders.
stefano
parents: 9428
diff changeset
95 sub->rects[0]->type = SUBTITLE_BITMAP;
8516
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
96 sub->rects[0]->pict.linesize[0] = w;
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
97 sub->rects[0]->pict.data[0] = av_malloc(w * h);
8512
aa45029f5cd7 Change AVSubtitle.rects to an array of pointers so ABI does not break
michael
parents: 7040
diff changeset
98 sub->rects[0]->nb_colors = 4;
10069
8ac9bc10b485 Always allocate a buffer of AVPALETTE_SIZE for palette in the subtitle
reimar
parents: 10054
diff changeset
99 sub->rects[0]->pict.data[1] = av_mallocz(AVPALETTE_SIZE);
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
100
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
101 // read palette
8512
aa45029f5cd7 Change AVSubtitle.rects to an array of pointers so ABI does not break
michael
parents: 7040
diff changeset
102 for (i = 0; i < sub->rects[0]->nb_colors; i++)
8516
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
103 ((uint32_t*)sub->rects[0]->pict.data[1])[i] = bytestream_get_be24(&buf);
5489
dc54869af30b Colours except background should not be transparent
reimar
parents: 5488
diff changeset
104 // make all except background (first entry) non-transparent
8512
aa45029f5cd7 Change AVSubtitle.rects to an array of pointers so ABI does not break
michael
parents: 7040
diff changeset
105 for (i = 1; i < sub->rects[0]->nb_colors; i++)
8516
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
106 ((uint32_t*)sub->rects[0]->pict.data[1])[i] |= 0xff000000;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
107
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
108 // process RLE-compressed data
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
109 rlelen = FFMIN(rlelen, buf_end - buf);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
110 init_get_bits(&gb, buf, rlelen * 8);
8516
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
111 bitmap = sub->rects[0]->pict.data[0];
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
112 for (y = 0; y < h; y++) {
5490
12d77ed34985 Deinterlace xsub subtitles
reimar
parents: 5489
diff changeset
113 // interlaced: do odd lines
8516
315b302fcd1d Replace AVSubtitleRect.rgba_palette and bitmap by AVPicture.
michael
parents: 8512
diff changeset
114 if (y == (h + 1) / 2) bitmap = sub->rects[0]->pict.data[0] + w;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
115 for (x = 0; x < w; ) {
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
116 int log2 = ff_log2_tab[show_bits(&gb, 8)];
5487
3039b660bf35 get rid of xsubdec array and calculate value instead
reimar
parents: 5486
diff changeset
117 int run = get_bits(&gb, 14 - 4 * (log2 >> 1));
5982
1900b70712ab colour --> color in variable names
diego
parents: 5493
diff changeset
118 int color = get_bits(&gb, 2);
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
119 run = FFMIN(run, w - x);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
120 // run length 0 means till end of row
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
121 if (!run) run = w - x;
5982
1900b70712ab colour --> color in variable names
diego
parents: 5493
diff changeset
122 memset(bitmap, color, run);
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
123 bitmap += run;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
124 x += run;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
125 }
5490
12d77ed34985 Deinterlace xsub subtitles
reimar
parents: 5489
diff changeset
126 // interlaced, skip every second line
12d77ed34985 Deinterlace xsub subtitles
reimar
parents: 5489
diff changeset
127 bitmap += w;
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
128 align_get_bits(&gb);
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
129 }
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
130 *data_size = 1;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
131 return buf_size;
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
132 }
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
133
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
134 AVCodec xsub_decoder = {
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
135 "xsub",
11560
8a4984c5cacc Define AVMediaType enum, and use it instead of enum CodecType, which
stefano
parents: 10069
diff changeset
136 AVMEDIA_TYPE_SUBTITLE,
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
137 CODEC_ID_XSUB,
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
138 0,
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
139 decode_init,
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
140 NULL,
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
141 NULL,
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
142 decode_frame,
7040
e943e1409077 Make AVCodec long_names definition conditional depending on CONFIG_SMALL.
stefano
parents: 6710
diff changeset
143 .long_name = NULL_IF_CONFIG_SMALL("XSUB"),
5483
447409d8770f First version of xsub decoder, not yet tested
reimar
parents:
diff changeset
144 };