glx segfault on glXSwapIntervalSGI with fglrx

added Component::Video: misc. plugins Severity::normal Type::bug Version::master git + 1 deleted label

changed milestone to %Bugs paradize

According to X11 debug symbols,

dpy is NULL when calling XInitExtension, which queries and tries to lock a NULL display.

Seems parameter-less glXSwapIntervalSGI is missing some earlier context initialization.


Thread 25 (Thread 0x7fffe0aca700 (LWP 19789)):
[#0](https://code.videolan.org/videolan/vlc/-/issues/0)  XQueryExtension (dpy=dpy@entry=0x0, name=name@entry=0x380b4965b2 "GLX", major_opcode=major_opcode@entry=0x7fffe0ac90d4, first_event=first_event@entry=0x7fffe0ac90d8, first_error=first_error@entry=0x7fffe0ac90dc) at QuExt.c:43
        rep = {type = 41 ')', pad1 = 48 '0', sequenceNumber = 0, length = 0, present = 32 ' ', major_opcode = 48 '0', first_event = 0 '\000', first_error = 0 '\000', pad3 = 32, pad4 = 12324, pad5 = 8, pad6 = 12323, pad7 = 8}
        req = <optimized out>
[#1](https://code.videolan.org/videolan/vlc/-/issues/1)  0x0000003806e2cee2 in XInitExtension (dpy=0x0, name=0x380b4965b2 "GLX") at InitExt.c:47
        codes = {extension = -871293424, major_opcode = 32767, first_event = -743967677, first_error = 56}
        ext = <optimized out>
[#2](https://code.videolan.org/videolan/vlc/-/issues/2)  0x000000380720d44d in XextAddDisplay () from /lib64/libXext.so.6
No symbol table info available.
[#3](https://code.videolan.org/videolan/vlc/-/issues/3)  0x000000380b440054 in ?? () from /usr/lib64/catalyst/libGL.so.1
No symbol table info available.
[#4](https://code.videolan.org/videolan/vlc/-/issues/4)  0x000000380b440f33 in ?? () from /usr/lib64/catalyst/libGL.so.1
No symbol table info available.
[#5](https://code.videolan.org/videolan/vlc/-/issues/5)  0x000000380b439d3f in glXSwapIntervalSGI () from /usr/lib64/catalyst/libGL.so.1
No symbol table info available.
[#6](https://code.videolan.org/videolan/vlc/-/issues/6)  0x00007fffba01d60c in Open (obj=0x7fffcc109fb8) at glx.c:196
        SwapIntervalSGI = <optimized out>
        gl = 0x7fffcc109fb8
        dpy = 0x7fffcc0008c0
        sys = 0x7fffcc108470
        wa = {x = 0, y = 0, width = 1024, height = 576, border_width = 0, depth = 24, visual = 0x7fffcc10fce0, root = 184, class = 1, bit_gravity = 1, win_gravity = 1, backing_store = 0, backing_planes = 4294967295, backing_pixel = 0, save_under = 0, colormap = 32, map_installed = 1, map_state = 2, all_event_masks = 6479987, your_event_mask = 0, do_not_propagate_mask = 0, override_redirect = 0, screen = 0x7fffcc1128b0}
        snum = 0
        visual = 35
        attr = {8, 5, 9, 5, 10, 5, 5, 1, 32786, 1, 32784, 1, 0}
        nelem = 34
        confs = 0x7fffcc388220
        conf = 0x7fffcc021750
        found = true
        is_swap_interval_set = false
        __PRETTY_FUNCTION__ = "Open"

http://www.opengl.org/registry/specs/EXT/swap_control.txt

_ Calling glXSwapIntervalSGI is equivalent to calling glXSwapIntervalEXT on the current drawable, if one exists._

-> Seems it needs a current Drawable, which does not exists yet in Open() before attaching it to current context using glXMakeContextCurrent()

Replying to [comment:2 fcartegnie]:

-> Seems it needs a current Drawable, which does not exists yet in Open() before attaching it to current context using glXMakeContextCurrent()

There are actually two separate bugs. OTOH, fglrx should return a no-context error instead of crashing. OTOH, VLC should associate the context first.

assigned to @Courmisch and unassigned @MigrationBot

added Component::Video: X11 Platform::GNU/Linux labels and removed Component::Video: misc. plugins + 1 deleted label

changed milestone to %2.1.x maintenance bugs

commit d79244f3 Author: Rémi Denis-Courmont remi@remlab.net Date: Thu May 30 18:42:53 2013 +0300

glx: make context current before setting swap interval (fixes [#8738](https://code.videolan.org/videolan/vlc/-/issues/8738))

Original author: remi@remlab.net

added Status::fixed label

closed

glx segfault on glXSwapIntervalSGI with fglrx

Child items 0

Activity