diff options
| author | Jamey Sharp <jamey@minilop.net> | 2006-10-15 00:03:57 -0700 |
|---|---|---|
| committer | Daniel Stone <daniels@endtroducing.fooishbar.org> | 2007-04-11 18:44:32 +0300 |
| commit | 0ae08175472acc203c55eaf886fa754901072108 (patch) | |
| tree | efd965408225586fdd3044340e0db59bbac1329b /src/XExtInt.c | |
| parent | a768f06e3c795279dfed289ea6544e2b4c302ae6 (diff) | |
Don't call XInput_find_display in _XiGetExtensionVersion, while the Display lock is held.
_XiGetExtensionVersion has two callers. One had already called
XInput_find_display, and the other could easily do so outside the
Display lock. So I change the _XiGetExtensionVersion interface to
accept a previously-computed XExtDisplayInfo from the caller.
Besides being more correct, this should be slightly faster. :-)
Thanks to Magnus Kessler <Magnus.Kessler@gmx.net> for identifying
the bug and proposing a workaround.
Diffstat (limited to 'src/XExtInt.c')
| -rw-r--r-- | src/XExtInt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/XExtInt.c b/src/XExtInt.c index f9f2f96..29765ef 100644 --- a/src/XExtInt.c +++ b/src/XExtInt.c @@ -211,7 +211,7 @@ _XiCheckExtInit(dpy, version_index, info) return (-1); } ((XInputData *) info->data)->vers = - _XiGetExtensionVersion(dpy, "XInputExtension"); + _XiGetExtensionVersion(dpy, "XInputExtension", info); } if (versions[version_index].major_version > Dont_Check) { |