Skip to content

Commit

Permalink
Fixes a bug in silk_sum_sqr_shift() that was causing issues with Burg.
Browse files Browse the repository at this point in the history
When an overflow happened in silk_sum_sqr_shift(), the sample pair causing
the overflow would be counted twice. This would lead to the C0 in
silk_burg_modified() being too large and cause problems on loud, tonal
signals. Thanks to Marcello Caramma for tracking down the problem.
  • Loading branch information
jmvalin committed Jun 18, 2014
1 parent f396c08 commit a88d836
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions silk/sum_sqr_shift.c
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ void silk_sum_sqr_shift(
/* Scale down */
nrg = (opus_int32)silk_RSHIFT_uint( (opus_uint32)nrg, 2 );
shft = 2;
i+=2;
break;
}
}
Expand Down

0 comments on commit a88d836

Please sign in to comment.