commit 2f09c72a625eb0fefd0dc2d255f913cf39d621ff
author Chih-hung Hsieh <chh@google.com> Wed Apr 22 18:16:57 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed Apr 22 18:16:57 2020 +0000
tree 1c010ef26e1e3bde75cf605acd95e2f09aabd7aa
parent 7aee64d4849fd1a69a3216d03b1db58781cd9ecf
parent 38f14dabfb69f43ffa56b788f22b93af713f8e55

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

Change-Id: I67e3f3af7a20251cd95ef733446d8104b9b01ba2

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";
 }