From 62b93e56e2e3c821b2c384cf201168ccd5059902 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. (cherry picked from commit 7506039f1b31eeaddab436831d23fb50c2f9409b) --- 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 a19e53dc2..a5808e7ce 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