[Mesa-dev] [Bug 95456] glXGetFBConfigs has invalid screen bounds

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed May 18 11:40:07 UTC 2016


https://bugs.freedesktop.org/show_bug.cgi?id=95456

--- Comment #1 from Stefan Dirsch <sndirsch at suse.de> ---
(In reply to Jiri Slaby from comment #0)
> Created attachment 123829 [details] [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 mesa-dev at lists.freedesktop.org, but it is moderated
> and I am not sure it went through.

It did. You even got a 'Reviewed-by' by ajax.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160518/ca8079bb/attachment.html>


More information about the mesa-dev mailing list