X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=libavcodec%2Fvorbis.c;h=66fa21b9e19dc3d881f0ee7a5dcb6bdd90eb2cf4;hb=b77fffa127663028169c5ed543956af4b9496c29;hp=5fa96c61861ea1fc1497a78d90f90d5edf1c0303;hpb=5e56b30e1febefd4eee7a0831ccbcb979cc11a2e;p=ffmpeg diff --git a/libavcodec/vorbis.c b/libavcodec/vorbis.c index 5fa96c61861..66fa21b9e19 100644 --- a/libavcodec/vorbis.c +++ b/libavcodec/vorbis.c @@ -1,29 +1,28 @@ -/** - * @file libavcodec/vorbis.c - * Common code for Vorbis I encoder and decoder - * @author Denes Balatoni ( dbalatoni programozo hu ) - - * This file is part of FFmpeg. +/* + * This file is part of Libav. * - * FFmpeg is free software; you can redistribute it and/or + * Libav is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public * License as published by the Free Software Foundation; either * version 2.1 of the License, or (at your option) any later version. * - * FFmpeg is distributed in the hope that it will be useful, + * Libav is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * Lesser General Public License for more details. * * You should have received a copy of the GNU Lesser General Public - * License along with FFmpeg; if not, write to the Free Software + * License along with Libav; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#undef V_DEBUG -//#define V_DEBUG +/** + * @file + * Common code for Vorbis I encoder and decoder + * @author Denes Balatoni ( dbalatoni programozo hu ) + */ -#define ALT_BITSTREAM_READER_LE +#define BITSTREAM_READER_LE #include "avcodec.h" #include "get_bits.h" @@ -35,12 +34,13 @@ // x^(1/n) unsigned int ff_vorbis_nth_root(unsigned int x, unsigned int n) { - unsigned int ret=0, i, j; + unsigned int ret = 0, i, j; do { ++ret; - for(i=0,j=ret;i 32 checks should be redundant, all calling code should // already ensure that, but since it allows overwriting the stack it seems // reasonable to check redundantly. -int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num) +int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, unsigned num) { - uint_fast32_t exit_at_level[33]={404,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, - 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}; + uint32_t exit_at_level[33] = { 404 }; - uint_fast8_t i,j; - uint_fast32_t code,p; + unsigned i, j, p, code; -#ifdef V_DEBUG +#ifdef DEBUG GetBitContext gb; #endif - for(p=0;(bits[p]==0) && (pavccontext, AV_LOG_INFO, "An empty codebook. Heh?! \n"); + for (p = 0; (bits[p] == 0) && (p < num); ++p) + ; + if (p == num) return 0; - } - codes[p]=0; - if (bits[p] > 32) return 1; - for(i=0;i 32) + return 1; + for (i = 0; i < bits[p]; ++i) + exit_at_level[i+1] = 1 << i; -#ifdef V_DEBUG - av_log(NULL, AV_LOG_INFO, " %d. of %d code len %d code %d - ", p, num, bits[p], codes[p]); - init_get_bits(&gb, (uint_fast8_t *)&codes[p], bits[p]); - for(i=0;i 32) return 1; - if (bits[p]==0) continue; + for (; p < num; ++p) { + if (bits[p] > 32) + return 1; + if (bits[p] == 0) + continue; // find corresponding exit(node which the tree can grow further from) - for(i=bits[p];i>0;--i) - if (exit_at_level[i]) break; - if (!i) return 1; // overspecified tree - code=exit_at_level[i]; - exit_at_level[i]=0; + for (i = bits[p]; i > 0; --i) + if (exit_at_level[i]) + break; + if (!i) // overspecified tree + return 1; + code = exit_at_level[i]; + exit_at_level[i] = 0; // construct code (append 0s to end) and introduce new exits - for(j=i+1;j<=bits[p];++j) - exit_at_level[j]=code+(1<<(j-1)); - codes[p]=code; + for (j = i + 1 ;j <= bits[p]; ++j) + exit_at_level[j] = code + (1 << (j - 1)); + codes[p] = code; -#ifdef V_DEBUG +#ifdef DEBUG av_log(NULL, AV_LOG_INFO, " %d. code len %d code %d - ", p, bits[p], codes[p]); - init_get_bits(&gb, (uint_fast8_t *)&codes[p], bits[p]); - for(i=0;i list[list[i].low].x) list[i].low = j; + if (tmp > list[list[i].low].x) + list[i].low = j; } else { - if (tmp < list[list[i].high].x) list[i].high = j; + if (tmp < list[list[i].high].x) + list[i].high = j; } } } for (i = 0; i < values - 1; i++) { int j; for (j = i + 1; j < values; j++) { + if (list[i].x == list[j].x) { + av_log(avctx, AV_LOG_ERROR, + "Duplicate value found in floor 1 X coordinates\n"); + return AVERROR_INVALIDDATA; + } if (list[list[i].sort].x > list[list[j].sort].x) { int tmp = list[i].sort; list[i].sort = list[j].sort; @@ -143,61 +154,62 @@ void ff_vorbis_ready_floor1_list(vorbis_floor1_entry * list, int values) } } } + return 0; } -static inline void render_line_unrolled(intptr_t x, intptr_t y, int x1, +static inline void render_line_unrolled(intptr_t x, int y, int x1, intptr_t sy, int ady, int adx, - float * buf) + float *buf) { int err = -adx; - x -= x1-1; - buf += x1-1; + x -= x1 - 1; + buf += x1 - 1; while (++x < 0) { err += ady; if (err >= 0) { err += ady - adx; - y += sy; - buf[x++] = ff_vorbis_floor1_inverse_db_table[y]; + y += sy; + buf[x++] = ff_vorbis_floor1_inverse_db_table[av_clip_uint8(y)]; } - buf[x] = ff_vorbis_floor1_inverse_db_table[y]; + buf[x] = ff_vorbis_floor1_inverse_db_table[av_clip_uint8(y)]; } if (x <= 0) { if (err + ady >= 0) y += sy; - buf[x] = ff_vorbis_floor1_inverse_db_table[y]; + buf[x] = ff_vorbis_floor1_inverse_db_table[av_clip_uint8(y)]; } } -static void render_line(int x0, int y0, int x1, int y1, float * buf) +static void render_line(int x0, int y0, int x1, int y1, float *buf) { - int dy = y1 - y0; + int dy = y1 - y0; int adx = x1 - x0; int ady = FFABS(dy); - int sy = dy<0 ? -1 : 1; - buf[x0] = ff_vorbis_floor1_inverse_db_table[y0]; - if(ady*2<=adx) { // optimized common case + int sy = dy < 0 ? -1 : 1; + buf[x0] = ff_vorbis_floor1_inverse_db_table[av_clip_uint8(y0)]; + if (ady*2 <= adx) { // optimized common case render_line_unrolled(x0, y0, x1, sy, ady, adx, buf); } else { - int base = dy / adx; - int x = x0; - int y = y0; - int err = -adx; + int base = dy / adx; + int x = x0; + int y = y0; + int err = -adx; ady -= FFABS(base) * adx; while (++x < x1) { y += base; err += ady; if (err >= 0) { err -= adx; - y += sy; + y += sy; } - buf[x] = ff_vorbis_floor1_inverse_db_table[y]; + buf[x] = ff_vorbis_floor1_inverse_db_table[av_clip_uint8(y)]; } } } void ff_vorbis_floor1_render_list(vorbis_floor1_entry * list, int values, - uint_fast16_t * y_list, int * flag, - int multiplier, float * out, int samples) + uint16_t *y_list, int *flag, + int multiplier, float *out, int samples) { int lx, ly, i; lx = 0; @@ -212,7 +224,9 @@ void ff_vorbis_floor1_render_list(vorbis_floor1_entry * list, int values, lx = x1; ly = y1; } - if (lx >= samples) break; + if (lx >= samples) + break; } - if (lx < samples) render_line(lx, ly, samples, ly, out); + if (lx < samples) + render_line(lx, ly, samples, ly, out); }