Re: [SyncEvolution] Segfault in sync-ui 1.3.2
On Mon, Sep 30, 2013 at 22:08:15 +0200, Patrick Ohly wrote:
On Mon, 2013-09-30 at 18:03 +0200,
tino.keitel+syncevolution(a)tikei.de
wrote:
> On Mon, Sep 30, 2013 at 17:09:29 +0200, tino.keitel+syncevolution(a)tikei.de wrote:
> > Hi,
> >
> > meanwhile, I could nail it down to the compiler/linker flags used for
> > the package build (export DEB_BUILD_MAINT_OPTIONS = hardening=+all).
> >
> > I'll report back what flag exactly causes the
> > segfault.
>
> The segfault happens when I build with -fPIE and link with -fPIE -pie.
> As this mainly deals with address space layout, my guess is that this
> just unhides another bug.
Bingo! You've found a 64 bit bug in sync-ui. As the other bug, this goes
back to the original development of the UI.
At one point, it passes a string pointer through a 32bit unsigned int,
Hi,
thanks, the segmentation fault is now gone.
When looking at the gdb trace, I thought that the "out of bounds"
address looks like there where 16 high bits missing compared to the
other addresses, so I also suspected a wrong pointer usage and the
patch perfectly makes sense.
Regards,
Tino