Handle GL_READ_BUFFER being GL_NONE in getReadAttachment()

Change-Id: If3883de8f49a3fb33ecc598ba0b86c658cd3f0ca Reviewed-on: https://chromium-review.googlesource.com/415565Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org>

Handle GL_READ_BUFFER being GL_NONE in getReadAttachment()
2ec65dc8 · Antoine Labour · Commit Bot · 47c0e048 · 2ec65dc8
Commit 2ec65dc8 authored Nov 30, 2016 by Antoine Labour Committed by Commit Bot Dec 01, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

Framebuffer.cpp src/libANGLE/Framebuffer.cpp +4 -0

No files found.
--- a/src/libANGLE/Framebuffer.cpp
+++ b/src/libANGLE/Framebuffer.cpp
@@ -96,6 +96,10 @@ const FramebufferAttachment *FramebufferState::getAttachment(GLenum attachment) 

 const FramebufferAttachment *FramebufferState::getReadAttachment() const
 {
+    if (mReadBufferState == GL_NONE)
+    {
+        return nullptr;
+    }
    ASSERT(mReadBufferState == GL_BACK || (mReadBufferState >= GL_COLOR_ATTACHMENT0 && mReadBufferState <= GL_COLOR_ATTACHMENT15));
    size_t readIndex = (mReadBufferState == GL_BACK ? 0 : static_cast<size_t>(mReadBufferState - GL_COLOR_ATTACHMENT0));
    ASSERT(readIndex < mColorAttachments.size());