annotate ra144dec.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 159554445343
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
1 /*
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
2 * Real Audio 1.0 (14.4K)
7163
ec5bae4d05a2 Update copyright
vitor
parents: 7162
diff changeset
3 *
ec5bae4d05a2 Update copyright
vitor
parents: 7162
diff changeset
4 * Copyright (c) 2008 Vitor Sessak
ec5bae4d05a2 Update copyright
vitor
parents: 7162
diff changeset
5 * Copyright (c) 2003 Nick Kurshev
ec5bae4d05a2 Update copyright
vitor
parents: 7162
diff changeset
6 * Based on public domain decoder at http://www.honeypot.net/audio
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
7 *
3947
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
8 * This file is part of FFmpeg.
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
9 *
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
10 * FFmpeg is free software; you can redistribute it and/or
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
11 * modify it under the terms of the GNU Lesser General Public
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
12 * License as published by the Free Software Foundation; either
3947
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
13 * version 2.1 of the License, or (at your option) any later version.
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
14 *
3947
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
15 * FFmpeg is distributed in the hope that it will be useful,
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
16 * but WITHOUT ANY WARRANTY; without even the implied warranty of
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
17 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
18 * Lesser General Public License for more details.
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
19 *
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
20 * You should have received a copy of the GNU Lesser General Public
3947
c8c591fe26f8 Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents: 3036
diff changeset
21 * License along with FFmpeg; if not, write to the Free Software
3036
0b546eab515d Update licensing information: The FSF changed postal address.
diego
parents: 2979
diff changeset
22 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
23 */
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
24
11417
421c9441c11e Move ff_sqrt() to libavutil/intmath.h
mru
parents: 11146
diff changeset
25 #include "libavutil/intmath.h"
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
26 #include "avcodec.h"
9428
0dce4fe6e6f3 Rename bitstream.h to get_bits.h.
stefano
parents: 9355
diff changeset
27 #include "get_bits.h"
1335
b4a72edb3a71 moved the tables into header files (and applied the 'static' patch). Nick: why do you like mergeing tables and code into one file, so making it unusable big?
al3x
parents: 1305
diff changeset
28 #include "ra144.h"
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
29
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
30
7828
0dc972a25a37 Add av_cold attribute to decode_init()
vitor
parents: 7827
diff changeset
31 static av_cold int ra144_decode_init(AVCodecContext * avctx)
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
32 {
6895
c63e9357e2a8 Context vars are not global vars. glob is a bad name for it.
vitor
parents: 6894
diff changeset
33 RA144Context *ractx = avctx->priv_data;
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
34
10776
816067b00019 Use correct context for av_log(), should prevent a crash for malformed files.
vitor
parents: 9428
diff changeset
35 ractx->avctx = avctx;
816067b00019 Use correct context for av_log(), should prevent a crash for malformed files.
vitor
parents: 9428
diff changeset
36
7128
614bc53e67a8 Use lpc_coef[2] instead of lpc_coef and lpc_coef_old
vitor
parents: 7127
diff changeset
37 ractx->lpc_coef[0] = ractx->lpc_tables[0];
614bc53e67a8 Use lpc_coef[2] instead of lpc_coef and lpc_coef_old
vitor
parents: 7127
diff changeset
38 ractx->lpc_coef[1] = ractx->lpc_tables[1];
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
39
7451
85ab7655ad4d Modify all codecs to report their supported input and output sample format(s).
pross
parents: 7447
diff changeset
40 avctx->sample_fmt = SAMPLE_FMT_S16;
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
41 return 0;
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
42 }
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
43
7133
6a25c2867e2f More cosmetics
vitor
parents: 7132
diff changeset
44 static void do_output_subblock(RA144Context *ractx, const uint16_t *lpc_coefs,
6a25c2867e2f More cosmetics
vitor
parents: 7132
diff changeset
45 int gval, GetBitContext *gb)
6832
9584a95fefcd Cosmetics: move function to remove forward declarations
vitor
parents: 6831
diff changeset
46 {
6907
8b0db8b07a21 cosmetics: typo fixes
diego
parents: 6902
diff changeset
47 int cba_idx = get_bits(gb, 7); // index of the adaptive CB, 0 if none
6900
9333cbdc4524 Rename var: add read coefficients a decent name
vitor
parents: 6899
diff changeset
48 int gain = get_bits(gb, 8);
9333cbdc4524 Rename var: add read coefficients a decent name
vitor
parents: 6899
diff changeset
49 int cb1_idx = get_bits(gb, 7);
9333cbdc4524 Rename var: add read coefficients a decent name
vitor
parents: 6899
diff changeset
50 int cb2_idx = get_bits(gb, 7);
6832
9584a95fefcd Cosmetics: move function to remove forward declarations
vitor
parents: 6831
diff changeset
51
11868
159554445343 Split do_output_subblock() into common code and decoder specific parts
vitor
parents: 11866
diff changeset
52 ff_subblock_synthesis(ractx, lpc_coefs, cba_idx, cb1_idx, cb2_idx, gval,
159554445343 Split do_output_subblock() into common code and decoder specific parts
vitor
parents: 11866
diff changeset
53 gain);
6832
9584a95fefcd Cosmetics: move function to remove forward declarations
vitor
parents: 6831
diff changeset
54 }
9584a95fefcd Cosmetics: move function to remove forward declarations
vitor
parents: 6831
diff changeset
55
7847
5a114f24632a misc spelling/wording/grammar fixes
diego
parents: 7834
diff changeset
56 /** Uncompress one block (20 bytes -> 160*2 bytes). */
7132
22788bf6502a Cosmetics
vitor
parents: 7128
diff changeset
57 static int ra144_decode_frame(AVCodecContext * avctx, void *vdata,
9355
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 9081
diff changeset
58 int *data_size, AVPacket *avpkt)
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
59 {
9355
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 9081
diff changeset
60 const uint8_t *buf = avpkt->data;
54bc8a2727b0 Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
rbultje
parents: 9081
diff changeset
61 int buf_size = avpkt->size;
6805
8faad14a40a5 Read the bitstream where the data is actually needed instead of doing it in unpack_input()
vitor
parents: 6804
diff changeset
62 static const uint8_t sizes[10] = {6, 5, 5, 4, 4, 3, 3, 3, 3, 2};
7084
424cb385c5ae Misc cosmetics
vitor
parents: 7083
diff changeset
63 unsigned int refl_rms[4]; // RMS of the reflection coefficients
11146
e8461dae9672 Fix reading beyond buffer end on RA144Context.lpc_tables.
vitor
parents: 10776
diff changeset
64 uint16_t block_coefs[4][10]; // LPC coefficients of each sub-block
7005
6e374f47da64 Remove *lpc_refl from the context. Only the value
vitor
parents: 7004
diff changeset
65 unsigned int lpc_refl[10]; // LPC reflection coefficients of the frame
7447
16737361e9ab Cosmetics: rename loop counter vars to i,j
vitor
parents: 7164
diff changeset
66 int i, j;
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
67 int16_t *data = vdata;
6898
27b34839f1db Rename var: val -> energy
vitor
parents: 6897
diff changeset
68 unsigned int energy;
6806
e4be053e7d9b Remove unneeded fields from the decoder context
vitor
parents: 6805
diff changeset
69
6934
ac6f00bed029 Revert r13499, log:
vitor
parents: 6928
diff changeset
70 RA144Context *ractx = avctx->priv_data;
6805
8faad14a40a5 Read the bitstream where the data is actually needed instead of doing it in unpack_input()
vitor
parents: 6804
diff changeset
71 GetBitContext gb;
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
72
7506
e94202a2e0b6 Check available size before writing in decode_frame()
vitor
parents: 7451
diff changeset
73 if (*data_size < 2*160)
e94202a2e0b6 Check available size before writing in decode_frame()
vitor
parents: 7451
diff changeset
74 return -1;
e94202a2e0b6 Check available size before writing in decode_frame()
vitor
parents: 7451
diff changeset
75
6854
d4c8338a09a8 Handle the case where we do not have enough input
vitor
parents: 6852
diff changeset
76 if(buf_size < 20) {
d4c8338a09a8 Handle the case where we do not have enough input
vitor
parents: 6852
diff changeset
77 av_log(avctx, AV_LOG_ERROR,
d4c8338a09a8 Handle the case where we do not have enough input
vitor
parents: 6852
diff changeset
78 "Frame too small (%d bytes). Truncated file?\n", buf_size);
7085
f55f71e2be57 Do not write junk in the end of truncated files. FATE
vitor
parents: 7084
diff changeset
79 *data_size = 0;
6854
d4c8338a09a8 Handle the case where we do not have enough input
vitor
parents: 6852
diff changeset
80 return buf_size;
d4c8338a09a8 Handle the case where we do not have enough input
vitor
parents: 6852
diff changeset
81 }
6805
8faad14a40a5 Read the bitstream where the data is actually needed instead of doing it in unpack_input()
vitor
parents: 6804
diff changeset
82 init_get_bits(&gb, buf, 20 * 8);
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
83
6805
8faad14a40a5 Read the bitstream where the data is actually needed instead of doing it in unpack_input()
vitor
parents: 6804
diff changeset
84 for (i=0; i<10; i++)
11866
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
85 lpc_refl[i] = ff_lpc_refl_cb[i][get_bits(&gb, sizes[i])];
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
86
11866
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
87 ff_eval_coefs(ractx->lpc_coef[0], lpc_refl);
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
88 ractx->lpc_refl_rms[0] = ff_rms(lpc_refl);
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
89
11866
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
90 energy = ff_energy_tab[get_bits(&gb, 5)];
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
91
11866
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
92 refl_rms[0] = ff_interp(ractx, block_coefs[0], 1, 1, ractx->old_energy);
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
93 refl_rms[1] = ff_interp(ractx, block_coefs[1], 2,
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
94 energy <= ractx->old_energy,
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
95 ff_t_sqrt(energy*ractx->old_energy) >> 12);
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
96 refl_rms[2] = ff_interp(ractx, block_coefs[2], 3, 0, energy);
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
97 refl_rms[3] = ff_rescale_rms(ractx->lpc_refl_rms[0], energy);
6968
50dd3a733f43 The function dec1() do two completly unrelated things. Split the rms calculation out of it.
vitor
parents: 6967
diff changeset
98
11866
0d8f35836b20 Add ff_ prefix to non-static functions
vitor
parents: 11865
diff changeset
99 ff_int_to_int16(block_coefs[3], ractx->lpc_coef[0]);
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
100
7447
16737361e9ab Cosmetics: rename loop counter vars to i,j
vitor
parents: 7164
diff changeset
101 for (i=0; i < 4; i++) {
16737361e9ab Cosmetics: rename loop counter vars to i,j
vitor
parents: 7164
diff changeset
102 do_output_subblock(ractx, block_coefs[i], refl_rms[i], &gb);
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
103
7447
16737361e9ab Cosmetics: rename loop counter vars to i,j
vitor
parents: 7164
diff changeset
104 for (j=0; j < BLOCKSIZE; j++)
16737361e9ab Cosmetics: rename loop counter vars to i,j
vitor
parents: 7164
diff changeset
105 *data++ = av_clip_int16(ractx->curr_sblock[j + 10] << 2);
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
106 }
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
107
6898
27b34839f1db Rename var: val -> energy
vitor
parents: 6897
diff changeset
108 ractx->old_energy = energy;
7128
614bc53e67a8 Use lpc_coef[2] instead of lpc_coef and lpc_coef_old
vitor
parents: 7127
diff changeset
109 ractx->lpc_refl_rms[1] = ractx->lpc_refl_rms[0];
6781
2f505f532aa8 Replace some hardcoded swapping with FFSWAP.
astrange
parents: 6780
diff changeset
110
7128
614bc53e67a8 Use lpc_coef[2] instead of lpc_coef and lpc_coef_old
vitor
parents: 7127
diff changeset
111 FFSWAP(unsigned int *, ractx->lpc_coef[0], ractx->lpc_coef[1]);
6781
2f505f532aa8 Replace some hardcoded swapping with FFSWAP.
astrange
parents: 6780
diff changeset
112
6828
0969192d5f8b Simplify ra144_decode_frame()
vitor
parents: 6826
diff changeset
113 *data_size = 2*160;
6767
f95c5091b802 cosmetics: Reindent file, patch by Vitor Sessak, vitor1001 gmail com,
diego
parents: 6712
diff changeset
114 return 20;
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
115 }
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
116
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
117 AVCodec ra_144_decoder =
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
118 {
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
119 "real_144",
11560
8a4984c5cacc Define AVMediaType enum, and use it instead of enum CodecType, which
stefano
parents: 11417
diff changeset
120 AVMEDIA_TYPE_AUDIO,
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
121 CODEC_ID_RA_144,
6894
eb3c16b99f27 Rename context struct to be more consistent with the rest of ffmpeg
vitor
parents: 6888
diff changeset
122 sizeof(RA144Context),
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
123 ra144_decode_init,
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
124 NULL,
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
125 NULL,
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
126 ra144_decode_frame,
7040
e943e1409077 Make AVCodec long_names definition conditional depending on CONFIG_SMALL.
stefano
parents: 7005
diff changeset
127 .long_name = NULL_IF_CONFIG_SMALL("RealAudio 1.0 (14.4K)"),
1304
e8543aab0cc9 RealAudio 1.0 (14_4) and 2.0 (28_8) native decoders. Author unknown, code from a mplayerhq (originally from public domain player for Amiga
nickols_k
parents:
diff changeset
128 };