#39 bug in missile launch code

[Rene Stoeckel]

There was a jake2 crash recently in the game engine. It
failed with a nullpointer exception while the berserk
monster tried to launch a missile. This looks like a bug
coming from quake2. Quake2 seems to handle these
things with a setjmp, in jake2 this protection is not
working yet, but should be there in the next release.

[Rene Stoeckel]

stack trace

[Holger Zickner]

Logged In: YES
user_id=594249

This is not a problem of setjmp/longjmp handling. These are
used for error handling in Q2 but there is no error handling
for wrong or missing function pointers in Q2. The problem
seems to be an incorrect attack_state. ai_run_missile()
should never be called for a berserk as both of his attacks
are triggered by ai_run_melee(). ai_checkattack() decides
which attack method to call dependent on attack_state
(AS_MISSILE or AS_MELEE). It could be an error assigning the
wrong attack_state somewhere.