Re: BUGFIX

From: d. hall (dhall@APK.NET)
Date: 01/16/98


// thus on Fri, 16 Jan 1998 09:26:42 -0500, George virtually wrote:

> On Fri, 16 Jan 1998, Chuck Reed wrote:
>> Hi.  The bugfix george posted crashes my game instead of preventing the
>> multiple players.  It hangs in the if (GET_NAME(dt->character) &&
>> !str_cmp(GET_NAME(dt->character), newname)) line.  Anyone else have this
>> happen, and if so, any ideas on how to fix it?

> What is line 252 (the GN(dc) && !sc(GN(dc), nw) line?

> If it does die, print out '*dt' and '*dt->character'.  I didn't have a
> problem, although perhaps your dt->character doesn't exist while mine did.
> (although I haven't changed anything in that section of code in the code
> base I was using)

Based upon nanny() in interpreter.c, dt->character MUST exist before
Valid_name() is called, which is one of the reasons why when I posted
previously the blurb, I didn't check for character.

Maybe an "assert (dt->character);" before the check, so you can do a
#define NDEBUG after you know it works.  The only other reason is that
GET_NAME() checks if IS_NPC(), so... if that character structure has the
NPC bit set, it may cause problems, with a backtrace, it's hard to say.

d.


     +------------------------------------------------------------+
     | Ensure that you have read the CircleMUD Mailing List FAQ:  |
     | http://democracy.queensu.ca/~fletcher/Circle/list-faq.html |
     +------------------------------------------------------------+



This archive was generated by hypermail 2b30 : 12/15/00 PST