From 3ab0d92ac918ade820e0f62b21e6bc691a8b62ef Mon Sep 17 00:00:00 2001 From: SHIRAKATA Kentaro Date: Wed, 9 Feb 2022 18:37:36 +0900 Subject: [PATCH] remove unnecessary condition on save_mtraits() `mtmp->data` here is always non-null, otherwise it leads segv at earlier code. --- src/mkobj.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mkobj.c b/src/mkobj.c index 3c1d21ad9..d94458e80 100644 --- a/src/mkobj.c +++ b/src/mkobj.c @@ -2052,8 +2052,7 @@ save_mtraits(struct obj *obj, struct monst *mtmp) *mtmp2 = *mtmp; mtmp2->mextra = (struct mextra *) 0; - if (mtmp->data) - mtmp2->mnum = monsndx(mtmp->data); + mtmp2->mnum = monsndx(mtmp->data); /* invalidate pointers */ /* m_id is needed to know if this is a revived quest leader */ /* but m_id must be cleared when loading bones */