commit 95587b085e5605bec6cebf7758d05731b384dbd3
author Treehugger Robot <treehugger-gerrit@google.com> Fri Mar 29 03:52:46 2019 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Mar 29 03:52:46 2019 +0000
tree 25ba5d14f9743f84d09b2d35b7b6dc26ef8f1586
parent 32a8a47b070047fd0970db2b3f4cf6b87fdf8565
parent ebe293ab54472f5adf6189edbaa1e73939320b66

Merge "Fix bogus error checking on unique_fd"

Checkpoint.cpp

diff --git a/Checkpoint.cpp b/Checkpoint.cpp
index e060230..8605b6a 100644
--- a/Checkpoint.cpp
+++ b/Checkpoint.cpp
@@ -330,7 +330,7 @@
         if (fstab_rec->fs_mgr_flags.checkpoint_blk) {
             android::base::unique_fd fd(
                 TEMP_FAILURE_RETRY(open(mount_rec.mount_point.c_str(), O_RDONLY | O_CLOEXEC)));
-            if (!fd) {
+            if (fd == -1) {
                 PLOG(ERROR) << "Failed to open mount point" << mount_rec.mount_point;
                 continue;
             }