The Static Analysis tool Coverity added that pointer stackAddress_ is not initialized in the constructor in the following case: >> case CodeRange::Inline: { >> // The throw stub clears WasmActivation::fp on it's way out. >> if (!fp) { >> MOZ_ASSERT(done()); >> return; >> } We can see that pointer is initiliazed at the end of the switch(codeRange->kind()) block: >> codeRange_ = codeRange; >> stackAddress_ = state.sp; >> MOZ_ASSERT(!done()); It can be seen there that also pointer codeRange_ is attributed a value that was previously initialized at the beginning of the contructor. Even though this could not be a problem we could still initialized pointer stackAddress_ in the constructor same as codeRange_ with a nullptr.

Review commit: https://reviewboard.mozilla.org/r/31421/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/31421/