[Nouveau] [Bug 66255] Enabling Xinerama with nouveau driver causes Segmentation fault

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jul 1 09:07:46 PDT 2013


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

--- Comment #14 from Trev Jackson <trev.g7pvs at ntlworld.com> ---
Hi

I'm not sure about this - I've not done any C programming for a long time, the
bit of code below from panoramiXprocs.c line 1837 where it has:
y < -wBorderWidth((WindowPtr)pDraw)
is there supposed to be a gap between the - symbol and the function?
wBorderWidth call and if so is that likely to cause a problem?

Trev

      if( /* check for being onscreen */
screenInfo.screens[0]->x + pDraw->x + x < 0 ||
screenInfo.screens[0]->x + pDraw->x + x + w > PanoramiXPixWidth ||
screenInfo.screens[0]->y + pDraw->y + y < 0 ||
screenInfo.screens[0]->y + pDraw->y + y + h > PanoramiXPixHeight ||
/* check for being inside of border */
        x < - wBorderWidth((WindowPtr)pDraw) ||
x + w > wBorderWidth((WindowPtr)pDraw) + (int)pDraw->width ||
y < -wBorderWidth((WindowPtr)pDraw) ||
y + h > wBorderWidth ((WindowPtr)pDraw) + (int)pDraw->height)
return BadMatch;
    }

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20130701/c8e7bc02/attachment.html>


More information about the Nouveau mailing list