diff options
| -rw-r--r-- | services/surfaceflinger/SurfaceFlinger.cpp | 8 | ||||
| -rw-r--r-- | services/surfaceflinger/SurfaceFlinger_hwc1.cpp | 8 |
2 files changed, 12 insertions, 4 deletions
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp index db03c23b36..3a0e21fb76 100644 --- a/services/surfaceflinger/SurfaceFlinger.cpp +++ b/services/surfaceflinger/SurfaceFlinger.cpp @@ -3440,14 +3440,18 @@ 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; } - // queueBuffer takes ownership of syncFd - result = window->queueBuffer(window, buffer, syncFd); + if (buffer) { + // queueBuffer takes ownership of syncFd + result = window->queueBuffer(window, buffer, syncFd); + } } } else { result = BAD_VALUE; diff --git a/services/surfaceflinger/SurfaceFlinger_hwc1.cpp b/services/surfaceflinger/SurfaceFlinger_hwc1.cpp index 16332f34cd..d864874a55 100644 --- a/services/surfaceflinger/SurfaceFlinger_hwc1.cpp +++ b/services/surfaceflinger/SurfaceFlinger_hwc1.cpp @@ -3478,14 +3478,18 @@ 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; } - // queueBuffer takes ownership of syncFd - result = window->queueBuffer(window, buffer, syncFd); + if (buffer) { + // queueBuffer takes ownership of syncFd + result = window->queueBuffer(window, buffer, syncFd); + } } } else { result = BAD_VALUE; |