Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4b39a60c authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Fix return types of sendMessageOnSocket/receiveMessageFromSocket" am:... 

Merge "Fix return types of sendMessageOnSocket/receiveMessageFromSocket" am: b1815e3c am: e7854e1f

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2269886



Change-Id: I12515d4e46d0538dc9d67fef8da04fd81408c26d
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents a35525e0 e7854e1f

libs/binder/OS.cpp

+2 −2
Original line number Diff line number Diff line
@@ -67,7 +67,7 @@ std::unique_ptr<RpcTransportCtxFactory> makeDefaultRpcTransportCtxFactory() { 
    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 @@ int sendMessageOnSocket( 
    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) {

libs/binder/OS.h

+2 −2
Original line number Diff line number Diff line
@@ -33,11 +33,11 @@ status_t dupFileDescriptor(int oldFd, int* newFd); 


std::unique_ptr<RpcTransportCtxFactory> makeDefaultRpcTransportCtxFactory();



int sendMessageOnSocket(

ssize_t sendMessageOnSocket(

        const RpcTransportFd& socket, iovec* iovs, int niovs,

        const std::vector<std::variant<base::unique_fd, base::borrowed_fd>>* ancillaryFds);



int receiveMessageFromSocket(

ssize_t receiveMessageFromSocket(

        const RpcTransportFd& socket, iovec* iovs, int niovs,

        std::vector<std::variant<base::unique_fd, base::borrowed_fd>>* ancillaryFds);

libs/binder/RpcTransportRaw.cpp

+2 −1
Original line number Diff line number Diff line
@@ -61,7 +61,8 @@ public: 
            override {

        bool sentFds = false;

        auto send = [&](iovec* iovs, int niovs) -> ssize_t {

            int ret = sendMessageOnSocket(mSocket, iovs, niovs, sentFds ? nullptr : ancillaryFds);

            ssize_t ret =

                    sendMessageOnSocket(mSocket, iovs, niovs, sentFds ? nullptr : ancillaryFds);

            sentFds |= ret > 0;

            return ret;

        };

libs/binder/trusty/OS.cpp

+2 −2
Original line number Diff line number Diff line
@@ -59,14 +59,14 @@ std::unique_ptr<RpcTransportCtxFactory> makeDefaultRpcTransportCtxFactory() { 
    return RpcTransportCtxFactoryTipcTrusty::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 */) {

    errno = ENOTSUP;

    return -1;

}



int receiveMessageFromSocket(

ssize_t receiveMessageFromSocket(

        const RpcTransportFd& /* socket */, iovec* /* iovs */, int /* niovs */,

        std::vector<std::variant<base::unique_fd, base::borrowed_fd>>* /* ancillaryFds */) {

    errno = ENOTSUP;