Use the proper lower bound when doing saturation arithmetic.
authorSean Bright <sean@malleable.com>
Tue, 30 Apr 2013 13:48:12 +0000 (13:48 +0000)
committerSean Bright <sean@malleable.com>
Tue, 30 Apr 2013 13:48:12 +0000 (13:48 +0000)
16 bit signed integers have a range of [-32768, 32768).  The existing code
was using the interval (-32768, 32768) instead.  This patch fixes that.

Review: https://reviewboard.asterisk.org/r/2479/
........

Merged revisions 386929 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 386930 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@386931 65c4cc65-6c06-0410-ace0-fbb531ad65f3

include/asterisk/utils.h

index 9452dd5..6f041f9 100644 (file)
@@ -324,8 +324,8 @@ static force_inline void ast_slinear_saturated_add(short *input, short *value)
        res = (int) *input + *value;
        if (res > 32767)
                *input = 32767;
-       else if (res < -32767)
-               *input = -32767;
+       else if (res < -32768)
+               *input = -32768;
        else
                *input = (short) res;
 }
@@ -337,8 +337,8 @@ static force_inline void ast_slinear_saturated_subtract(short *input, short *val
        res = (int) *input - *value;
        if (res > 32767)
                *input = 32767;
-       else if (res < -32767)
-               *input = -32767;
+       else if (res < -32768)
+               *input = -32768;
        else
                *input = (short) res;
 }
@@ -350,8 +350,8 @@ static force_inline void ast_slinear_saturated_multiply(short *input, short *val
        res = (int) *input * *value;
        if (res > 32767)
                *input = 32767;
-       else if (res < -32767)
-               *input = -32767;
+       else if (res < -32768)
+               *input = -32768;
        else
                *input = (short) res;
 }