diff options
| author | 2010-12-22 22:39:18 +0100 | |
|---|---|---|
| committer | 2012-01-13 20:57:27 +0200 | |
| commit | 51a50f01b598c538fc415b00f488827724f6668e (patch) | |
| tree | ee4cdecf2f531dbfb25ec08d9b9426d33f676c0e | |
| parent | b4cee0f871e0f5a2abd6c6f05b65710b69423f2c (diff) | |
stagefright aacenc: Don't address a Word32 with a Word16 pointer
Currently, a pointer to a Word32 is used as a pointer to Word16, which
doesn't work as intended on big endian architectures.
This change needs to be analyzed and tested properly so that it doesn't
cause issues with overflows in these variables. Another solution would be
to change all the scf arrays to 32 bit instead.
Change-Id: I53d9eb83bdaf8d3b1530eb5064dd10b08bce737a
| -rw-r--r-- | media/libstagefright/codecs/aacenc/src/sf_estim.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/libstagefright/codecs/aacenc/src/sf_estim.c b/media/libstagefright/codecs/aacenc/src/sf_estim.c index fe401374c022..bc320ec67df7 100644 --- a/media/libstagefright/codecs/aacenc/src/sf_estim.c +++ b/media/libstagefright/codecs/aacenc/src/sf_estim.c @@ -400,7 +400,7 @@ static void assimilateSingleScf(PSY_OUT_CHANNEL *psyOutChan, Word16 *minScfCalculated, Flag restartOnSuccess) { - Word32 sfbLast, sfbAct, sfbNext, scfAct, scfMin; + Word16 sfbLast, sfbAct, sfbNext, scfAct, scfMin; Word16 *scfLast, *scfNext; Word32 sfbPeOld, sfbPeNew; Word32 sfbDistNew; |