Fix some 16-bit int issues in the multistream API.
authorTimothy B. Terriberry <tterribe@xiph.org>
Mon, 18 Mar 2013 21:42:44 +0000 (14:42 -0700)
committerTimothy B. Terriberry <tterribe@xiph.org>
Mon, 18 Mar 2013 21:42:44 +0000 (14:42 -0700)
With 120 ms frames and 6 or more channels, the total number of
 samples in the buffer could overflow.

src/opus_multistream_decoder.c
src/opus_multistream_encoder.c

index 47f87b1..495fb07 100644 (file)
@@ -274,7 +274,7 @@ static void opus_copy_channel_out_float(
 )
 {
    float *float_dst;
-   int i;
+   opus_int32 i;
    float_dst = (float*)dst;
    if (src != NULL)
    {
@@ -303,7 +303,7 @@ static void opus_copy_channel_out_short(
 )
 {
    opus_int16 *short_dst;
-   int i;
+   opus_int32 i;
    short_dst = (opus_int16*)dst;
    if (src != NULL)
    {
index 66c75ca..1847bd5 100644 (file)
@@ -356,7 +356,7 @@ static void opus_copy_channel_in_float(
 )
 {
    const float *float_src;
-   int i;
+   opus_int32 i;
    float_src = (const float *)src;
    for (i=0;i<frame_size;i++)
 #if defined(FIXED_POINT)
@@ -377,7 +377,7 @@ static void opus_copy_channel_in_short(
 )
 {
    const opus_int16 *short_src;
-   int i;
+   opus_int32 i;
    short_src = (const opus_int16 *)src;
    for (i=0;i<frame_size;i++)
 #if defined(FIXED_POINT)