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

Commit 937e1bdc authored by Tom Cherry's avatar Tom Cherry
Browse files

Use sizeof(android_log_header_t) instead of sizeof its components 

Particularly because sizeof_log_id_t is a misnomer and is going away.

Test: build, liblog and logcat unit tests
Change-Id: I7978bf83e44336a334229753ff1e11a07134e8fc
parent 0d5b27b2

cmds/statsd/src/socket/StatsSocketListener.cpp

+1 −2
Original line number Diff line number Diff line
@@ -56,8 +56,7 @@ bool StatsSocketListener::onDataAvailable(SocketClient* cli) { 
    }



    // + 1 to ensure null terminator if MAX_PAYLOAD buffer is received

    char buffer[sizeof_log_id_t + sizeof(uint16_t) + sizeof(log_time) + LOGGER_ENTRY_MAX_PAYLOAD +

                1];

    char buffer[sizeof(android_log_header_t) + LOGGER_ENTRY_MAX_PAYLOAD + 1];

    struct iovec iov = {buffer, sizeof(buffer) - 1};



    alignas(4) char control[CMSG_SPACE(sizeof(struct ucred))];