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

Commit ff48c151 authored by Prathmesh Prabhu's avatar Prathmesh Prabhu Committed by android-build-merger
Browse files

Merge "Do not queue trivial buffer for unsupported screenshot request." into nyc-dev 

am: c334549a

* commit 'c334549a':
  Do not queue trivial buffer for unsupported screenshot request.
parents ebfa3dba c334549a

services/surfaceflinger/SurfaceFlinger.cpp

+6 −2
Original line number Diff line number Diff line
@@ -3440,15 +3440,19 @@ status_t SurfaceFlinger::captureScreenImplLocked( 
                    } else {

                        ALOGE("got GL_FRAMEBUFFER_COMPLETE_OES error while taking screenshot");

                        result = INVALID_OPERATION;

                        window->cancelBuffer(window, buffer, syncFd);

                        buffer = NULL;

                    }

                    // destroy our image

                    eglDestroyImageKHR(mEGLDisplay, image);

                } else {

                    result = BAD_VALUE;

                }

                if (buffer) {

                    // queueBuffer takes ownership of syncFd

                    result = window->queueBuffer(window, buffer, syncFd);

                }

            }

        } else {

            result = BAD_VALUE;

        }

services/surfaceflinger/SurfaceFlinger_hwc1.cpp

+6 −2
Original line number Diff line number Diff line
@@ -3478,15 +3478,19 @@ status_t SurfaceFlinger::captureScreenImplLocked( 
                    } else {

                        ALOGE("got GL_FRAMEBUFFER_COMPLETE_OES error while taking screenshot");

                        result = INVALID_OPERATION;

                        window->cancelBuffer(window, buffer, syncFd);

                        buffer = NULL;

                    }

                    // destroy our image

                    eglDestroyImageKHR(mEGLDisplay, image);

                } else {

                    result = BAD_VALUE;

                }

                if (buffer) {

                    // queueBuffer takes ownership of syncFd

                    result = window->queueBuffer(window, buffer, syncFd);

                }

            }

        } else {

            result = BAD_VALUE;

        }