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

Commit b7ec41f5 authored by Keith Mok's avatar Keith Mok Committed by Automerger Merge Worker
Browse files

Merge "Improper EINTR handling logic" am: bd0a86f9 am: 1b910626 

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1891585

Change-Id: Ib1d729a0affeec1b4c0fa2004b490a3e0e6a86e6
parents 413011e1 1b910626

packages/PrintSpooler/jni/com_android_printspooler_util_BitmapSerializeUtils.cpp

+8 −13
Original line number Diff line number Diff line
@@ -30,11 +30,8 @@ static bool writeAllBytes(const int fd, void* buffer, const size_t byteCount) { 
    char* writeBuffer = static_cast<char*>(buffer);

    size_t remainingBytes = byteCount;

    while (remainingBytes > 0) {

        ssize_t writtenByteCount = write(fd, writeBuffer, remainingBytes);

        ssize_t writtenByteCount = TEMP_FAILURE_RETRY(write(fd, writeBuffer, remainingBytes));

        if (writtenByteCount == -1) {

            if (errno == EINTR) {

                continue;

            }

            __android_log_print(ANDROID_LOG_ERROR, LOG_TAG,

                    "Error writing to buffer: %d", errno);

            return false;
@@ -49,19 +46,17 @@ static bool readAllBytes(const int fd, void* buffer, const size_t byteCount) { 
    char* readBuffer = static_cast<char*>(buffer);

    size_t remainingBytes = byteCount;

    while (remainingBytes > 0) {

        ssize_t readByteCount = read(fd, readBuffer, remainingBytes);



        remainingBytes -= readByteCount;

        readBuffer += readByteCount;



        ssize_t readByteCount = TEMP_FAILURE_RETRY(read(fd, readBuffer, remainingBytes));

        if (readByteCount == -1) {

            if (errno == EINTR) {

                continue;

            }

            __android_log_print(ANDROID_LOG_ERROR, LOG_TAG,

                    "Error reading from buffer: %d", errno);

            return false;

        } else if (readByteCount == 0 && remainingBytes > 0) {

        }



        remainingBytes -= readByteCount;

        readBuffer += readByteCount;



        if (readByteCount == 0 && remainingBytes > 0) {

            __android_log_print(ANDROID_LOG_ERROR, LOG_TAG,

                    "File closed before all bytes were read. %zu/%zu remaining", remainingBytes,

                    byteCount);