[Mesa3d-dev] [Bug 27628] New: glxgears prints bogus swap interval information

[<bug...@fr...>]

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

           Summary: glxgears prints bogus swap interval information
           Product: Mesa
           Version: git
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Demos
        AssignedTo: mes...@li...
        ReportedBy: msc...@re...


Created an attachment (id=34997)
 View: https://bugs.freedesktop.org/attachment.cgi?id=34997
 Review: https://bugs.freedesktop.org/review?bug=27628&attachment=34997

[PATCH] Initialize swap_interval to 0

$ glxgears 
Running synchronized to the vertical refresh.  The framerate should be
approximately 1/1920103026 the monitor refresh rate.

The number is obviously nonsense. This is on current Fedora 13
(mesa-7.8.1-1.fc13).

I looked into it and here's what I found:
 - glxgears gets this number using GLX_MESA_swap_control from
glXGetSwapIntervalMESA().
 - glXGetSwapIntervalMESA() calls into src/glx/dri2_glx.c:dri2GetSwapInterval()
which returns priv->swap_interval - this value is not initialized. Valgrind
agrees:

==25411== Conditional jump or move depends on uninitialised value(s)
==25411==    at 0x402C9A: main (glxgears.c:619)

Should pdraw->swap_interval be initialized to 0 in dri2CreateDrawable(), like
in the attached patch?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.