Summary: | Piglit/spec/EXT_framebuffer_object/fbo-bind-renderbuffer failed | ||
---|---|---|---|
Product: | Mesa | Reporter: | Samuel Iglesias Gonsálvez <siglesias> |
Component: | Mesa core | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | ||
Version: | git | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | Patch |
Description
Samuel Iglesias Gonsálvez
2014-04-01 09:32:31 UTC
Created attachment 96707 [details] [review] Patch This is the patch that changes the returned error value to GL_INVALID_OPERATION Fixed in master by: commit 9927180714662456ff7b895221f67112f2567a53 Author: Samuel Iglesias Gonsalvez <siglesias@igalia.com> Date: Thu Apr 3 08:30:06 2014 +0200 mesa: fix check for dummy renderbuffer in _mesa_FramebufferRenderbufferEXT() According to the spec: <renderbuffertarget> must be RENDERBUFFER and <renderbuffer> should be set to the name of the renderbuffer object to be attached to the framebuffer. <renderbuffer> must be either zero or the name of an existing renderbuffer object of type <renderbuffertarget>, otherwise an INVALID_OPERATION error is generated. This patch changes the previous returned GL_INVALID_VALUE to GL_INVALID_OPERATION. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76894 Cc: mesa-stable@lists.freedesktop.org Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com> Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com> Thanks for finding and fixing this! |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.