Bug#700977: rott: segfault in Darian level

Luc Habert Luc.Habert at normalesup.org
Wed Feb 20 13:17:46 UTC 2013


On Wed, Feb 20, 2013 at 01:36:32PM +0100, Fabian Greffrath wrote:
> However, it turns out that only the seond chunk of your patch is actually
> required to prevent the crash.  If you look at the code in rt_actor.c,
> then tempstat is always set to some value (tempstat = sprites[tx][ty]; in
> line 8173) before touchsprite is set to it (touchsprite = tempstat; in
> line 8181).

It's strange, because gdb was showing me a non-null (but still pointing to
an invalid address) value for touchsprite, which led me to believe that it
was assigned from an uninitialized tempstat.  But looking at the code more
closely, I can only agree with you.  Actually, gdb's output was a bit weird
(for instance, the backtrace was incomplete), so it may be that it read
touchsprite's value at the wrong place too.



More information about the Pkg-games-devel mailing list