http://bugs.freedesktop.org/show_bug.cgi?id=13091


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]
             Status|NEW                         |RESOLVED
          Component|Drivers/DRI/i915            |GLX
         Resolution|                            |FIXED




------- Comment #4 from [EMAIL PROTECTED]  2007-11-26 18:38 PST -------

I found that when GLX create dricontext and make context current, it will fetch
a dri drawable and inserts this drawable into the glx Hash table. (code path:
MakeContextCurrent->FetchDRIDrawable-> __glxHashInsert).

When GLX destory dricontext, it will look up glx Hash table and get the dri
drawable and destroy it, but it forgets to delete the Hash entry!!!
So when glean finish second round testing on second visual, it will destory the
dri drawable, but it get the first dri drawable entry in hash table, which is
wrong.

Below is the patch, it will fix this bug. If it is OK, please commit it in.


--- a/src/glx/x11/glxcmds.c
+++ b/src/glx/x11/glxcmds.c
@@ -101,6 +101,7 @@ static void GarbageCollectDRIDrawables(D
                   longer exists in the Xserver */
                (*pdraw->driDrawable.destroyDrawable)(&pdraw->driDrawable);
                XF86DRIDestroyDrawable(dpy, sc->scr, draw);
+                __glxHashDelete(sc->drawHash, draw);
                Xfree(pdraw);
            }
        } while (__glxHashNext(sc->drawHash, &draw, (void *)&pdraw) == 1);


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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to