summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2019-10-31 17:13:50 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2019-10-31 17:13:50 +0000
commitf3229053656e57321eaeac81348db697d9f04859 (patch)
treee0ddc6e9d198d691a065821bf3764011116d4cec
parentc8b2e63c92a1cd6ff79b7e27afe0f860307f5969 (diff)
parente630555a7812731e040f862858060ce49d0d7b9f (diff)
downloadnetd-f3229053656e57321eaeac81348db697d9f04859.tar.gz
Merge "Enable pedantic checks for uninitialized variables"
-rw-r--r--Android.bp2
-rw-r--r--server/ResolvStub.cpp2
2 files changed, 3 insertions, 1 deletions
diff --git a/Android.bp b/Android.bp
index 114b2e16..97637304 100644
--- a/Android.bp
+++ b/Android.bp
@@ -10,6 +10,8 @@ cc_defaults {
"-Werror",
// Override -Wno-error=implicit-fallthrough from soong
"-Werror=implicit-fallthrough",
+ "-Werror=sometimes-uninitialized",
+ "-Werror=conditional-uninitialized",
"-Wnullable-to-nonnull-conversion",
"-Wsign-compare",
"-Wthread-safety",
diff --git a/server/ResolvStub.cpp b/server/ResolvStub.cpp
index d6bcb05a..2fb66254 100644
--- a/server/ResolvStub.cpp
+++ b/server/ResolvStub.cpp
@@ -55,7 +55,7 @@ static void resolvStubInitFunction(void* handle, const char* symbol, FunctionTyp
}
int resolv_stub_init() {
- void* netdResolvHandle;
+ void* netdResolvHandle = nullptr;
for (const auto& dir : {APEX_LIB64_DIR, APEX_LIB_DIR}) {
std::string path = std::string(dir) + "/" + LIBNAME;