Fix CopyBufferSubData validation.

We were checking readTarget twice, instead of both read and write. BUG=angle:800 Change-Id: I4096a9c4e0fdec9c77bfc25250923c188d2b43db Reviewed-on: https://chromium-review.googlesource.com/224102Reviewed-by: Geoff Lang <geofflang@chromium.org> Tested-by: Jamie Madill <jmadill@chromium.org>

Fix CopyBufferSubData validation.
24e83192 · Jamie Madill · 0661fce4 · 24e83192
Commit 24e83192 authored Oct 20, 2014 by Jamie Madill
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

libGLESv2.cpp src/libGLESv2/libGLESv2.cpp +1 -1

No files found.
--- a/src/libGLESv2/libGLESv2.cpp
+++ b/src/libGLESv2/libGLESv2.cpp
@@ -6993,7 +6993,7 @@ void GL_APIENTRY glCopyBufferSubData(GLenum readTarget, GLenum writeTarget, GLin
            return;
        }

-        if (!gl::ValidBufferTarget(context, readTarget) || !gl::ValidBufferTarget(context, readTarget))
+        if (!gl::ValidBufferTarget(context, readTarget) || !gl::ValidBufferTarget(context, writeTarget))
        {
            context->recordError(gl::Error(GL_INVALID_ENUM));
            return;