Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Path: utzoo!utgpu!water!watmath!clyde!cbosgd!mandrill!hal!ncoast!crds@ncoast.UUCP
From: crds@ncoast.UUCP
Newsgroups: comp.sources.bugs
Subject: Re: Conquest dumps core!! -- Hahah lucky that's all!
Message-ID: <5550@ncoast.UUCP>
Date: Tue, 10-Nov-87 02:16:41 EST
Article-I.D.: ncoast.5550
Posted: Tue Nov 10 02:16:41 1987
Date-Received: Sat, 14-Nov-87 09:18:36 EST
Sender: crds@ncoast.UUCP
Distribution: comp
Organization: Cleveland Public Access UN*X, Cleveland, Oh
Lines: 24
Summary: Serious problems in three modules


I found major problems in the following modules (all related):

      combat.c    npc.c    update.c

The problem(s) is(are):  Arrays being accessed out of bounds, causing improper
updates and memory management violations (on my system it watches for these).
Some are not obvious bugs ie. for(var=1;var<MAXvar;var++) SHOULD start at 0,
and DOES in fact for some, but NOT all of the updates.... others are simply
not being checked ie. for(var1=0;var1<MAXvar1;var1++) for(var2=var1-1;var2<=
var1+1;var2++) and then references to vararray(var2,etc.etc.) which is a
problem for the first and last elements.  Too many to list specifically.  I
believe I have all three modules corrected properly (I spent several days); I
no longer get suspended processes at all, and conquest seems to work great.  I
am willing to post these modules in their updated form, if I get enough
requests; otherwise, happy hunting!

Glenn A. Emelko
crds@ncoast.UUCP

PS: Those of you who are claiming it works great -- I sure hope your machine
is paying attention to what is being accessed, you just might have a bigger
problem yet to materialize from running conquest (maybe something at the core
level -- gasp -- I hope not!).