commit 0fc006685c20134b754cd96f7589ea6f93d93cd2
author Josh Gao <jmgao@google.com> Sat Feb 20 02:39:51 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Sat Feb 20 02:39:51 2016 +0000
tree 9afabea8515eaf5343977753ab245a77a6841a8d
parent 44b08c722d05f650c9b6872e28c73affe68eb55a
parent 2275f7da7370d5b403a5d4a9d558c34a38ee5c94

Merge "adb: sysdeps_test: improve smoke test."

adb/sysdeps_test.cpp

diff --git a/adb/sysdeps_test.cpp b/adb/sysdeps_test.cpp
index 3904cc0..78efea8 100644
--- a/adb/sysdeps_test.cpp
+++ b/adb/sysdeps_test.cpp
@@ -122,12 +122,14 @@
 };
 
 TEST_F(sysdeps_poll, smoke) {
-    adb_pollfd pfd[2];
+    adb_pollfd pfd[2] = {};
     pfd[0].fd = fds[0];
     pfd[0].events = POLLRDNORM;
     pfd[1].fd = fds[1];
     pfd[1].events = POLLWRNORM;
 
+    pfd[0].revents = -1;
+    pfd[1].revents = -1;
     EXPECT_EQ(1, adb_poll(pfd, 2, 0));
     EXPECT_EQ(0, pfd[0].revents);
     EXPECT_EQ(POLLWRNORM, pfd[1].revents);
@@ -135,16 +137,18 @@
     ASSERT_TRUE(WriteFdExactly(fds[1], "foo", 4));
 
     // Wait for the socketpair to be flushed.
+    pfd[0].revents = -1;
     EXPECT_EQ(1, adb_poll(pfd, 1, 100));
     EXPECT_EQ(POLLRDNORM, pfd[0].revents);
-
+    pfd[0].revents = -1;
+    pfd[1].revents = -1;
     EXPECT_EQ(2, adb_poll(pfd, 2, 0));
     EXPECT_EQ(POLLRDNORM, pfd[0].revents);
     EXPECT_EQ(POLLWRNORM, pfd[1].revents);
 }
 
 TEST_F(sysdeps_poll, timeout) {
-    adb_pollfd pfd;
+    adb_pollfd pfd = {};
     pfd.fd = fds[0];
     pfd.events = POLLRDNORM;
 
@@ -158,7 +162,7 @@
 }
 
 TEST_F(sysdeps_poll, invalid_fd) {
-    adb_pollfd pfd[3];
+    adb_pollfd pfd[3] = {};
     pfd[0].fd = fds[0];
     pfd[0].events = POLLRDNORM;
     pfd[1].fd = INT_MAX;
@@ -179,7 +183,7 @@
 }
 
 TEST_F(sysdeps_poll, duplicate_fd) {
-    adb_pollfd pfd[2];
+    adb_pollfd pfd[2] = {};
     pfd[0].fd = fds[0];
     pfd[0].events = POLLRDNORM;
     pfd[1] = pfd[0];
@@ -196,7 +200,7 @@
 }
 
 TEST_F(sysdeps_poll, disconnect) {
-    adb_pollfd pfd;
+    adb_pollfd pfd = {};
     pfd.fd = fds[0];
     pfd.events = POLLIN;