commit 62aef7d364670c9c255f8470b0ea874ac2cca293
author Andy Hung <hunga@google.com> Thu Dec 14 14:50:40 2017 -0800
committer Andy Hung <hunga@google.com> Thu Dec 14 16:15:39 2017 -0800
tree a3a0a713b480a85ea4fbaae85dbbb7b80f211808
parent ab30516bf9e5f371b720f9cdcd6adfc11e278648

EffectModule: initialize all member variables in constructor

Test: Solo Tester and CTS effect tests
Bug: 70674145
Change-Id: I9a9e137b8c612f0ea8862da3973d142636fad34d

services/audioflinger/Effects.cpp

diff --git a/services/audioflinger/Effects.cpp b/services/audioflinger/Effects.cpp
index 5d04a5b..cd3c588 100644
--- a/services/audioflinger/Effects.cpp
+++ b/services/audioflinger/Effects.cpp
@@ -72,9 +72,10 @@
       // prior to configure().
       mConfig{{}, {}},
       mStatus(NO_INIT), mState(IDLE),
-      // mMaxDisableWaitCnt is set by configure() and not used before then
-      // mDisableWaitCnt is set by process() and updateState() and not used before then
+      mMaxDisableWaitCnt(1), // set by configure(), should be >= 1
+      mDisableWaitCnt(0),    // set by process() and updateState()
       mSuspended(false),
+      mOffloaded(false),
       mAudioFlinger(thread->mAudioFlinger)
 #ifdef FLOAT_EFFECT_CHAIN
       , mSupportsFloat(false)