Mercurial > hg > early-roguelike

diff arogue7/sticks.c @ 314:2f0eb38da609

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Advanced Rogue 7: fix a crash when casting magic missile. do_zap() dereferenced a struct object pointer while ignoring the result of a NULL check. XRogue had the same problem, though triggering it was unlikely, since XRogue does not include a magic missile spell. Reported by John Harris of @Play.
author John "Elwin" Edwards
date Thu, 21 Oct 2021 21:00:15 -0400
parents e52a8a7ad4c5
children
line wrap: on
line diff
--- a/arogue7/sticks.c	Tue Oct 19 20:39:00 2021 -0400
+++ b/arogue7/sticks.c	Thu Oct 21 21:00:15 2021 -0400
@@ -309,7 +309,7 @@
 
 	    if (!obj)
 		dice = zapper->t_stats.s_lvl;
-	    if (obj->o_type == RELIC)
+	    else if (obj->o_type == RELIC)
 		 dice = 15;
 	    else if (EQUAL(ws_type[which], "staff"))
 		 dice = 10;