commit 4830902acd6bdf380e525f2fe87a64a78a844710
author Nick Maniscalco <maniscalco@google.com> Thu Mar 21 19:32:58 2019 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Thu Mar 21 19:32:58 2019 +0000
tree d5af37ae39249aa789545728b28b7690b32d1210
parent 9a0c508a31f6480f70f8c7e08c44b9d0c4d12b7d

[zircon][block] Initialize ServerManager::state_

Initialize ServerManager::state_ to None to ensure we don't
read uninitialized memory if server creation fails.

Bug: ZX-3758 #done
Change-Id: If76a14165b52db5822066e09fec2853c0addacba

zircon/system/dev/block/core/server-manager.h

diff --git a/zircon/system/dev/block/core/server-manager.h b/zircon/system/dev/block/core/server-manager.h
index c09ccf7..5f2b4b7 100644
--- a/zircon/system/dev/block/core/server-manager.h
+++ b/zircon/system/dev/block/core/server-manager.h
@@ -68,14 +68,14 @@
     static int RunServer(void* arg);
 
     ThreadState GetState() const {
-        return static_cast<ThreadState>(state_.load());
+        return state_.load();
     }
 
     void SetState(ThreadState state) {
-        state_.store(static_cast<uint32_t>(state));
+        state_.store(state);
     }
 
     thrd_t thread_;
-    std::atomic<uint32_t> state_;
+    std::atomic<ThreadState> state_ = ThreadState::None;
     BlockServer* server_ = nullptr;
 };