Re: winex11: Don't trace a garbage value or read past end of caller's array in X11DRV_wglChoosePixelFormatARB().

2013-09-20 Thread Ken Thomases
On Sep 20, 2013, at 9:44 AM, Roderick Colenbrander wrote: > That change looks suspicious. If switching to i+1 fixes a problem, the real > problem is that the if-statement in question gets entered in the first > place. Some bounds checking on the outer for-loop (the one which loops > through nCfgs

Re: winex11: Don't trace a garbage value or read past end of caller's array in X11DRV_wglChoosePixelFormatARB().

2013-09-20 Thread Roderick Colenbrander
That change looks suspicious. If switching to i+1 fixes a problem, the real problem is that the if-statement in question gets entered in the first place. Some bounds checking on the outer for-loop (the one which loops through nCfgs and nMaxFormats), is probably wrong then. I had issues with this pa