commit 38f14dabfb69f43ffa56b788f22b93af713f8e55
author Chih-hung Hsieh <chh@google.com> Wed Apr 22 18:03:34 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed Apr 22 18:03:34 2020 +0000
tree 442dcb0f27af322a965a4c788439eac278f6fe44
parent 6b30653d44fc35a8a6f4ff9bd48b2c8c98062b06
parent 168b2f5ce2c4ed559a7e79b70e185f96936cd273

Merge "Fix bugprone-posix-return warnings" am: 2816b20196 am: 5a8fd4c4c1 am: 46760802b4 am: 168b2f5ce2

Change-Id: I6cb61d679ec3eb4f03a72f89df71cb6575586a2d

media/mtp/MtpFfsHandle.cpp

diff --git a/media/mtp/MtpFfsHandle.cpp b/media/mtp/MtpFfsHandle.cpp
index bd6a6c6..c8b4a03 100644
--- a/media/mtp/MtpFfsHandle.cpp
+++ b/media/mtp/MtpFfsHandle.cpp
@@ -114,11 +114,11 @@
 void MtpFfsHandle::advise(int fd) {
     for (unsigned i = 0; i < NUM_IO_BUFS; i++) {
         if (posix_madvise(mIobuf[i].bufs.data(), MAX_FILE_CHUNK_SIZE,
-                POSIX_MADV_SEQUENTIAL | POSIX_MADV_WILLNEED) < 0)
+                POSIX_MADV_SEQUENTIAL | POSIX_MADV_WILLNEED) != 0)
             PLOG(ERROR) << "Failed to madvise";
     }
     if (posix_fadvise(fd, 0, 0,
-                POSIX_FADV_SEQUENTIAL | POSIX_FADV_NOREUSE | POSIX_FADV_WILLNEED) < 0)
+                POSIX_FADV_SEQUENTIAL | POSIX_FADV_NOREUSE | POSIX_FADV_WILLNEED) != 0)
         PLOG(ERROR) << "Failed to fadvise";
 }