summaryrefslogtreecommitdiff
path: root/payload_consumer/file_descriptor.cc
diff options
context:
space:
mode:
authorandroid-build-team Robot <android-build-team-robot@google.com>2021-05-14 01:09:55 +0000
committerandroid-build-team Robot <android-build-team-robot@google.com>2021-05-14 01:09:55 +0000
commita6c543c64b135f521cae645c2f882d78bf7307e8 (patch)
tree3e0ef8551a610e593102d8ff0f0a2d8c9b682581 /payload_consumer/file_descriptor.cc
parent3d4e6e0d7ed2b609e0fe2b4d0fcb250808f97476 (diff)
parent422e893875d4d1c8cf73db5d39a0eac19bce3065 (diff)
Snap for 7360053 from 422e893875d4d1c8cf73db5d39a0eac19bce3065 to sc-release
Change-Id: I9d292ff0b5cbdc3e0ab778bcb3f9eca70d8a0277
Diffstat (limited to 'payload_consumer/file_descriptor.cc')
-rw-r--r--payload_consumer/file_descriptor.cc4
1 files changed, 3 insertions, 1 deletions
diff --git a/payload_consumer/file_descriptor.cc b/payload_consumer/file_descriptor.cc
index 7c69c1b4..da76327c 100644
--- a/payload_consumer/file_descriptor.cc
+++ b/payload_consumer/file_descriptor.cc
@@ -139,7 +139,9 @@ bool EintrSafeFileDescriptor::Flush() {
}
bool EintrSafeFileDescriptor::Close() {
- CHECK_GE(fd_, 0);
+ if (fd_ < 0) {
+ return false;
+ }
// https://stackoverflow.com/questions/705454/does-linux-guarantee-the-contents-of-a-file-is-flushed-to-disc-after-close
// |close()| doesn't imply |fsync()|, we need to do it manually.
fsync(fd_);