| Summary: | Instead of returning and error, glXMakeContextCurrent crashes | ||
|---|---|---|---|
| Product: | Mesa | Reporter: | Ian Romanick <idr> |
| Component: | GLX | Assignee: | mesa-dev |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | CC: | palglowr |
| Version: | git | ||
| Hardware: | Other | ||
| OS: | All | ||
| URL: | https://bugzilla.novell.com/show_bug.cgi?id=466806 | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | Generate an error if either the read or draw drawables are not GLXWindow | ||
|
Description
Ian Romanick
2009-01-16 16:58:21 UTC
Created attachment 22045 [details] [review] Generate an error if either the read or draw drawables are not GLXWindow Proposed fix. It turns out there was another bug in that code. GLX_ALIAS can only be used if the two functions are EXACT aliases. However, MakeContextCurrent and glXMakeContextCurrent have different sets of parameters! The only part of this that I'm unsure of is generating the X error. We only do that in one or two other places for direct rendering, but it seems like we should do it everywhere. Opinions? dups or deps to http://bugs.freedesktop.org/show_bug.cgi?id=18367 (In reply to comment #2) > dups or deps to > http://bugs.freedesktop.org/show_bug.cgi?id=18367 > I posted a reply to that bug. I don't think it's a dup. There's something else weird going on there. Fixed by commit ab9b4e1f593b2006a61c0f5440c2d7d3af80ea74. Mass version move, cvs -> git |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.