VNC patch for Xserver 1.3
Alan Hourihane
alanh at fairlite.demon.co.uk
Tue Oct 23 07:45:05 PDT 2007
On Tue, 2007-10-23 at 16:27 +0200, Stefan Dirsch wrote:
> On Tue, Oct 23, 2007 at 01:42:08PM +0300, Daniel Stone wrote:
> > On Tue, Oct 23, 2007 at 12:34:34PM +0200, Stefan Dirsch wrote:
> > > On Mon, Oct 22, 2007 at 10:12:27PM +0200, Stefan Dirsch wrote:
> > > > AFAICS the pInfo structure is filled correctly in xf86rfbMouseInit(),
> > > > but the content gets lost again in InitInput() before calling
> > > > xf86ActivateDevice(). :-(
> > >
> > > Obviously type_name/always_core_feedback, conf_idev/drv, and
> > > options/history_size are swapped on x86_64.
> >
> > Sure it's being built with _XSERVER64? All external modules require
> > xorg-server.h.
>
> Meanwhile I figured out that it is *not* set. A patch, which fixes
> this issue for me, is attached. Some background:
>
> * fixes segfault on 64bit during Xserver start; make sure to define
> _XSERVER64 by having HAVE_DIX_CONFIG_H defined and therefore
> including dix-config.h, so Atom is CARD32 instead of unsigned long
> before and no longer messes up the pInfo structure in
> xf86rfbMouseInit/xf86rfbKeybInit
Thanks Stefan.
Committed.
Alan.
More information about the xorg
mailing list