commit 8fffea9d605d7d87a8ab33c889eb9719c1e955ec
author Kelvin Zhang <zhangkelvin@google.com> Tue Jul 25 20:34:26 2023 -0700
committer Kelvin Zhang <zhangkelvin@google.com> Tue Jul 25 20:36:00 2023 -0700
tree 4761fb97efb051409054d3132c17e624b567dbe7
parent eede4c8305070905417fbf6d6d2902a49546e8d8

Lint: Initialize variables, log variables on error

Test: th
Change-Id: I47127424de63f708f099b9884f4823366d15a838

payload_consumer/verified_source_fd.cc

diff --git a/payload_consumer/verified_source_fd.cc b/payload_consumer/verified_source_fd.cc
index 09ac95b..3f17ad7 100644
--- a/payload_consumer/verified_source_fd.cc
+++ b/payload_consumer/verified_source_fd.cc
@@ -159,7 +159,9 @@
   source_fd_ = std::make_shared<EintrSafeFileDescriptor>();
   if (source_fd_ == nullptr)
     return false;
-  TEST_AND_RETURN_FALSE_ERRNO(source_fd_->Open(source_path_.c_str(), O_RDONLY));
+  if (!source_fd_->Open(source_path_.c_str(), O_RDONLY)) {
+    PLOG(ERROR) << "Failed to open " << source_path_;
+  }
   return true;
 }