diff options
author | Marco Nelissen <marcone@google.com> | 2012-10-17 10:57:02 -0700 |
---|---|---|
committer | Marco Nelissen <marcone@google.com> | 2012-10-17 10:57:02 -0700 |
commit | a4fe9a13c432afc5d1e94a4617b75e8f962540e9 (patch) | |
tree | 378f10b114a866dbe5feb3eaf7ee35f0a33f3aa2 /media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c | |
parent | bbe11e46faf8f9e3296431993c36d3efc61337b5 (diff) | |
download | frameworks_av-a4fe9a13c432afc5d1e94a4617b75e8f962540e9.zip frameworks_av-a4fe9a13c432afc5d1e94a4617b75e8f962540e9.tar.gz frameworks_av-a4fe9a13c432afc5d1e94a4617b75e8f962540e9.tar.bz2 |
Fix valgrind issues
We were reading some uninitialized memory when creating bass boost and EQ
effects, and using memcpy() with identical source and destination.
Change-Id: I15ea1b2c52ae05cbf54aef04351e89805e0ebf8e
Diffstat (limited to 'media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c')
-rw-r--r-- | media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c b/media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c index e83e515..c4767a8 100644 --- a/media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c +++ b/media/libeffects/lvm/lib/Eq/src/LVEQNB_Init.c @@ -264,6 +264,9 @@ LVEQNB_ReturnStatus_en LVEQNB_Init(LVEQNB_Handle_t *phInstance, MemSize = (pCapabilities->MaxBands * sizeof(LVEQNB_BandDef_t)); pInstance->pBandDefinitions = (LVEQNB_BandDef_t *)InstAlloc_AddMember(&AllocMem, MemSize); + // clear all the bands, setting their gain to 0, otherwise when applying new params, + // it will compare against uninitialized values + memset(pInstance->pBandDefinitions, 0, MemSize); MemSize = (pCapabilities->MaxBands * sizeof(LVEQNB_BiquadType_en)); pInstance->pBiquadType = (LVEQNB_BiquadType_en *)InstAlloc_AddMember(&AllocMem, MemSize); |