Skip to content

Commit

Permalink
eas_mdls: fix OOB read.
Browse files Browse the repository at this point in the history
Bug: 34031018
AOSP-Change-Id: I8d373c905f64286b23ec819bdbee51368b12e85a

CVE-2017-0541

Change-Id: Ifb1825e25751e98b7f1d5355c5d3d0699ec08be7
(cherry picked from commit 56d1532)
(cherry picked from commit f7b1abeb6bfd9eb404be641ea347cf2b1198140e)
  • Loading branch information
Wei Jia authored and andi34 committed Apr 30, 2017
1 parent ba00921 commit 5e0e9db
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion arm-wt-22k/lib_src/eas_mdls.c
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,8 @@
/* this define allows us to use the sndlib.h structures as RW memory */
#define SCNST

#include "log/log.h"

#include "eas_data.h"
#include "eas_host.h"
#include "eas_mdls.h"
Expand Down Expand Up @@ -2086,8 +2088,11 @@ static EAS_RESULT PushcdlStack (EAS_U32 *pStack, EAS_INT *pStackPtr, EAS_U32 val
{

/* stack overflow, return an error */
if (*pStackPtr >= CDL_STACK_SIZE)
if (*pStackPtr >= (CDL_STACK_SIZE - 1)) {
ALOGE("b/34031018, stackPtr(%d)", *pStackPtr);
android_errorWriteLog(0x534e4554, "34031018");
return EAS_ERROR_FILE_FORMAT;
}

/* push the value onto the stack */
*pStackPtr = *pStackPtr + 1;
Expand Down

0 comments on commit 5e0e9db

Please sign in to comment.