Bug#443822: I've found a fix (for: hex-a-hop: Got bt from a segfault :D)

[Gerfried Fuchs]

* Gerfried Fuchs <rhonda at deb.at> [2008-11-22 18:23:12 CET]:
> * Jens Seidel <jensseidel at users.sf.net> [2008-11-22 11:28:12 CET]:
> > I have access to i386, mips, mipsel only. I'm sure valgrind would help
> > (valgrind hex-a-hop) but this doesn't support exotic architectures and
> > I could not reproduce this bug with it in the past.
> 
>  valgrind does run it, but extremely slow. And I mean slow. Given that
> the bug doesn't happen always at the same stage and can't be narrowed
> down enough this is really painful ...

 Hmm, there is one message it spits out over and over again though:

==10317== Source and destination overlap in memcpy(0x48AE628, 0x48AE62C, 2556)
==10317==    at 0xFFBCF94: memcpy (mc_replace_strmem.c:402)
==10317==    by 0x872B024E: ???
==10317==    by 0xFF11D10: (within /usr/lib/libSDL-1.2.so.0.11.1)
==10317==    by 0xFF29468: SDL_LowerBlit (in /usr/lib/libSDL-1.2.so.0.11.1)
==10317==    by 0xFF29680: SDL_UpperBlit (in /usr/lib/libSDL-1.2.so.0.11.1)
==10317==    by 0x1002148C: HexPuzzle::Render() (hex_puzzzle.cpp:2629)
==10317==    by 0x10004260: main (gfx.cpp:402)

 That line in hex_puzzzle.cpp says:
    SDL_BlitSurface(screen, &src, screen, &dst);

 It's the only call to that function with twice the same argument
(screen) in the whole source - might that be anything important?

 So long,
Rhonda