<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - glXGetFBConfigs has invalid screen bounds" href="https://bugs.freedesktop.org/show_bug.cgi?id=95456#c1">Comment # 1</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - glXGetFBConfigs has invalid screen bounds" href="https://bugs.freedesktop.org/show_bug.cgi?id=95456">bug 95456</a> from <a class="email" href="mailto:sndirsch@suse.de" title="Stefan Dirsch <sndirsch@suse.de>"> Stefan Dirsch</a> <pre>(In reply to Jiri Slaby from <a href="show_bug.cgi?id=95456#c0">comment #0</a>) > Created <a href="attachment.cgi?id=123829" name="attach_123829" title="patch">attachment 123829</a> <a href="attachment.cgi?id=123829&action=edit" title="patch">[details]</a> <a href='page.cgi?id=splinter.html&bug=95456&attachment=123829'>[review]</a> [review] > patch > > This is bad: > if (priv && (priv->screens != NULL) > && (screen >= 0) && (screen <= ScreenCount(dpy)) > It should write: > if (priv && (priv->screens != NULL) > && (screen >= 0) && (screen < ScreenCount(dpy)) > > > I have sent a patch to <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a>, but it is moderated > and I am not sure it went through. It did. You even got a 'Reviewed-by' by ajax.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>