If a spell is continued via the NPC handler callback, double-check that it is really waiting for a script. This fixes #755 and #733.

1 files changed, 17 insertions, 1 deletions

diff --git a/src/map/magic-stmt.c b/src/map/magic-stmt.c
index 4c866ca..cba0659 100644
--- a/src/map/magic-stmt.c
+++ b/src/map/magic-stmt.c
@@ -1402,10 +1402,15 @@ spell_execute_d(invocation_t *invocation, int allow_deletion)
         spell_update_location(invocation);
         delta = spell_run(invocation, allow_deletion);
 
-        if (delta > 0)
+        if (delta > 0) {
+                if (invocation->timer) {
+                        fprintf(stderr, "[magic] FATAL ERROR: Trying to add multiple timers to the same spell! Already had timer: %d\n", invocation->timer);
+                        /* *((int *)0x0) = 0; */
+                }
                 invocation->timer = add_timer(gettick() + delta,
                                               &invocation_timer_callback,
                                               invocation->bl.id, 0);
+        }
 
         /* If 0, the script cleaned itself.  If -1 (wait-for-script), we must wait for the user. */
 }
@@ -1416,6 +1421,17 @@ spell_execute(invocation_t *invocation)
         spell_execute_d(invocation, 1);
 }
 
+void
+spell_execute_script(invocation_t *invocation)
+{
+        if (invocation->script_pos)
+                spell_execute_d(invocation, 1);
+        /* Otherwise the script-within-the-spell has been terminated by some other means.
+         * In practice this happens when the script doesn't wait for user input: the client
+         * may still notify the server that it's done.  Without the above check, we'd be
+         * running the same spell twice! */
+}
+
 int
 spell_attack(int caster_id, int target_id)
 {