diff options
| author | Jean-Michel Trivi <jmtrivi@google.com> | 2017-11-03 19:10:04 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2017-11-03 19:10:04 +0000 |
| commit | f6100335b880b080b5bfe58803213e71326a806a (patch) | |
| tree | 5f396ececb9b91799524fd02dec63064df4e34d6 /libFDK/include | |
| parent | 858c2a7c6ef54c80ca7c19bd274c7819dcc29398 (diff) | |
| parent | 9fb4261c43a2d15f3b77a7e56470ed6784f83d04 (diff) | |
| download | fdk-aac-f6100335b880b080b5bfe58803213e71326a806a.tar.gz fdk-aac-f6100335b880b080b5bfe58803213e71326a806a.tar.bz2 fdk-aac-f6100335b880b080b5bfe58803213e71326a806a.zip | |
DO NOT MERGE Prevent out of bound memory access in GetInvInt
am: 9fb4261c43
Change-Id: If0378d7935b9f651f7096eee8916d4f83889a629
Diffstat (limited to 'libFDK/include')
| -rw-r--r-- | libFDK/include/fixpoint_math.h | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/libFDK/include/fixpoint_math.h b/libFDK/include/fixpoint_math.h index 0d50f0a..6aa0a90 100644 --- a/libFDK/include/fixpoint_math.h +++ b/libFDK/include/fixpoint_math.h @@ -479,15 +479,19 @@ inline FIXP_DBL fAddSaturate(const FIXP_DBL a, const FIXP_DBL b) /** * \brief Calculate the value of 1/i where i is a integer value. It supports - * input values from 1 upto 80. + * input values from 0 upto 79. * \param intValue Integer input value. * \param FIXP_DBL representation of 1/intValue */ inline FIXP_DBL GetInvInt(int intValue) { - FDK_ASSERT((intValue > 0) && (intValue < 80)); - FDK_ASSERT(intValue<80); - return invCount[intValue]; + FDK_ASSERT((intValue >= 0) && (intValue < 80)); + if (intValue > 79) + return invCount[79]; + else if (intValue < 0) + return invCount[0]; + else + return invCount[intValue]; } |