commit c3964f68e251e3a80f952e2014b2b5dc738aff4c
author David Brazdil <dbrazdil@google.com> Mon Oct 24 23:06:14 2022 +0100
committer David Brazdil <dbrazdil@google.com> Tue Oct 25 20:52:39 2022 +0100
tree 34d96aa4018f0bf347803cace44b9a0ec0ddd0e5
parent 77f3da77c9aa5d7bb504ee72da5a21314f28eb9e

Fix return types of sendMessageOnSocket/receiveMessageFromSocket

recvmsg() and sendmsg() return ssize_t but the wrapper functions
returned int. Fix the return type.

Bug: 250685929
Test: atest rpcBinderTest
Change-Id: Ia3f7f58e286f24a8711aed97d9bc2b3cf6b78f0d

libs/binder/OS.cpp

diff --git a/libs/binder/OS.cpp b/libs/binder/OS.cpp
index 77e401f..ce60e33 100644
--- a/libs/binder/OS.cpp
+++ b/libs/binder/OS.cpp
@@ -67,7 +67,7 @@
     return RpcTransportCtxFactoryRaw::make();
 }
 
-int sendMessageOnSocket(
+ssize_t sendMessageOnSocket(
         const RpcTransportFd& socket, iovec* iovs, int niovs,
         const std::vector<std::variant<base::unique_fd, base::borrowed_fd>>* ancillaryFds) {
     if (ancillaryFds != nullptr && !ancillaryFds->empty()) {
@@ -112,7 +112,7 @@
     return TEMP_FAILURE_RETRY(sendmsg(socket.fd.get(), &msg, MSG_NOSIGNAL));
 }
 
-int receiveMessageFromSocket(
+ssize_t receiveMessageFromSocket(
         const RpcTransportFd& socket, iovec* iovs, int niovs,
         std::vector<std::variant<base::unique_fd, base::borrowed_fd>>* ancillaryFds) {
     if (ancillaryFds != nullptr) {