From 7506039f1b31eeaddab436831d23fb50c2f9409b Mon Sep 17 00:00:00 2001 From: Markus Armbruster Date: Wed, 7 Apr 2010 23:35:25 +0200 Subject: [PATCH] Fix test for capability engineer in lnd_hit_mine() Bogus array index, unpredictable result, can crash the server. Broken in commit ef7ea893, v4.3.24. --- src/lib/subs/lndsub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/subs/lndsub.c b/src/lib/subs/lndsub.c index 2f5134e86..4b0cec14b 100644 --- a/src/lib/subs/lndsub.c +++ b/src/lib/subs/lndsub.c @@ -862,7 +862,7 @@ lnd_hit_mine(struct lndstr *lp) nreport(lp->lnd_own, N_LHIT_MINE, 0, 1); m = MINE_LDAMAGE(); - if (lchr[lp->lnd_uid].l_flags & L_ENGINEER) + if (lchr[lp->lnd_type].l_flags & L_ENGINEER) m /= 2; landdamage(lp, m); -- 2.43.0