diff options
author | android-build-team Robot <android-build-team-robot@google.com> | 2021-05-14 01:09:55 +0000 |
---|---|---|
committer | android-build-team Robot <android-build-team-robot@google.com> | 2021-05-14 01:09:55 +0000 |
commit | a6c543c64b135f521cae645c2f882d78bf7307e8 (patch) | |
tree | 3e0ef8551a610e593102d8ff0f0a2d8c9b682581 /payload_consumer/file_descriptor.cc | |
parent | 3d4e6e0d7ed2b609e0fe2b4d0fcb250808f97476 (diff) | |
parent | 422e893875d4d1c8cf73db5d39a0eac19bce3065 (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.cc | 4 |
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_); |