commit 7153373983e86704d8f35914c4cefb8eadbd8aa4
author Mike McTernan <mikemcternan@google.com> Wed Aug 28 21:29:17 2024 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Aug 28 21:29:17 2024 +0000
tree b7463c166fa76f2b0348492b65d704786ce8917a
parent b2653d0b1326643a0d778f241c3779e82af664af
parent 733610f6835183cc67407280cc51af4b77273248

Merge "Mark the return of munmap() as unused." into main

libs/binder/Parcel.cpp

diff --git a/libs/binder/Parcel.cpp b/libs/binder/Parcel.cpp
index f30b2aa..925cb23 100644
--- a/libs/binder/Parcel.cpp
+++ b/libs/binder/Parcel.cpp
@@ -1725,7 +1725,9 @@
                 }
             }
         }
-        ::munmap(ptr, len);
+        if (::munmap(ptr, len) == -1) {
+            ALOGW("munmap() failed: %s", strerror(errno));
+        }
     }
     ::close(fd);
     return status;
@@ -3331,7 +3333,9 @@
 
 void Parcel::Blob::release() {
     if (mFd != -1 && mData) {
-        ::munmap(mData, mSize);
+        if (::munmap(mData, mSize) == -1) {
+            ALOGW("munmap() failed: %s", strerror(errno));
+        }
     }
     clear();
 }