Fix readBuffer validation

The argument to glReadBuffer should be validated against GL_MAX_COLOR_ATTACHMENTS, not GL_MAX_DRAW_BUFFERS. Bug: angleproject:4766 Change-Id: Ic184934bd01e68b936266eae69543ae66fd68ea3 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2252978Reviewed-by: Kenneth Russell <kbr@chromium.org> Reviewed-by: Jonah Ryan-Davis <jonahr@google.com> Commit-Queue: Kenneth Russell <kbr@chromium.org>

Fix readBuffer validation
18d412cb · James Darpinian · Commit Bot · 054aeeed · 18d412cb
Commit 18d412cb authored Jun 18, 2020 by James Darpinian Committed by Commit Bot Jun 25, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

validationES3.cpp src/libANGLE/validationES3.cpp +2 -2

No files found.
--- a/src/libANGLE/validationES3.cpp
+++ b/src/libANGLE/validationES3.cpp
@@ -1661,9 +1661,9 @@ bool ValidateReadBuffer(const Context *context, GLenum src)
    {
        GLuint drawBuffer = static_cast<GLuint>(src - GL_COLOR_ATTACHMENT0);

-        if (drawBuffer >= static_cast<GLuint>(context->getCaps().maxDrawBuffers))
+        if (drawBuffer >= static_cast<GLuint>(context->getCaps().maxColorAttachments))
        {
-            context->validationError(GL_INVALID_OPERATION, kExceedsMaxDrawBuffers);
+            context->validationError(GL_INVALID_OPERATION, kExceedsMaxColorAttachments);
            return false;
        }
    }