Re: ACMD(do_desc)

From: George Greer (greerga@circlemud.org)
Date: 05/12/02


On Sun, 12 May 2002, Joe Mann wrote:

>ACMD(do_desc)
>{
>  skip_spaces(&argument);
>  if (!*argument) {
[...]
>  else {
>    ch->player.description = argument;
[...]
>    }
>}

'argument' is a transient buffer.  As soon as the function ends it's
overwritten, making your description now point to random strings.  Try
using strdup()/free() to manage it.

You're just duplicating the code in nanny()::case#2 though.

--
George Greer
greerga@circlemud.org

--
   +---------------------------------------------------------------+
   | FAQ: http://qsilver.queensu.ca/~fletchra/Circle/list-faq.html |
   | Archives: http://post.queensu.ca/listserv/wwwarch/circle.html |
   | Newbie List:  http://groups.yahoo.com/group/circle-newbies/   |
   +---------------------------------------------------------------+



This archive was generated by hypermail 2b30 : 06/25/03 PDT