diff options
author | Aurelien Jacobs <aurel@gnuage.org> | 2008-09-30 02:22:46 +0400 |
---|---|---|
committer | Aurelien Jacobs <aurel@gnuage.org> | 2008-09-30 02:22:46 +0400 |
commit | 15ae1959d8c30db13e612db0d099ec106948cfb4 (patch) | |
tree | b930cf35c077760b01903bbf0beca058b19262a9 /libavcodec/atrac3.c | |
parent | 068e82bac15d5f389b523b14e87e2adc03aaa82e (diff) |
atrac3: ensure input frame is not overwritten (it is const)
this fixes the following warning:
atrac3.c:889: warning: assignment discards qualifiers from pointer target type
Originally committed as revision 15464 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec/atrac3.c')
-rw-r--r-- | libavcodec/atrac3.c | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/libavcodec/atrac3.c b/libavcodec/atrac3.c index 708627edbc..9a9f8ae3b6 100644 --- a/libavcodec/atrac3.c +++ b/libavcodec/atrac3.c @@ -778,11 +778,11 @@ static int decodeChannelSoundUnit (ATRAC3Context *q, GetBitContext *gb, channel_ * @param databuf the input data */ -static int decodeFrame(ATRAC3Context *q, uint8_t* databuf) +static int decodeFrame(ATRAC3Context *q, const uint8_t* databuf) { int result, i; float *p1, *p2, *p3, *p4; - uint8_t *ptr1, *ptr2; + uint8_t *ptr1; if (q->codingMode == JOINT_STEREO) { @@ -796,14 +796,20 @@ static int decodeFrame(ATRAC3Context *q, uint8_t* databuf) /* Framedata of the su2 in the joint-stereo mode is encoded in * reverse byte order so we need to swap it first. */ - ptr1 = databuf; - ptr2 = databuf+q->bytes_per_frame-1; + if (databuf == q->decoded_bytes_buffer) { + uint8_t *ptr2 = q->decoded_bytes_buffer+q->bytes_per_frame-1; + ptr1 = q->decoded_bytes_buffer; for (i = 0; i < (q->bytes_per_frame/2); i++, ptr1++, ptr2--) { FFSWAP(uint8_t,*ptr1,*ptr2); } + } else { + const uint8_t *ptr2 = databuf+q->bytes_per_frame-1; + for (i = 0; i < q->bytes_per_frame; i++) + q->decoded_bytes_buffer[i] = *ptr2--; + } /* Skip the sync codes (0xF8). */ - ptr1 = databuf; + ptr1 = q->decoded_bytes_buffer; for (i = 4; *ptr1 == 0xF8; i++, ptr1++) { if (i >= q->bytes_per_frame) return -1; @@ -875,7 +881,7 @@ static int atrac3_decode_frame(AVCodecContext *avctx, const uint8_t *buf, int buf_size) { ATRAC3Context *q = avctx->priv_data; int result = 0, i; - uint8_t* databuf; + const uint8_t* databuf; int16_t* samples = data; if (buf_size < avctx->block_align) |