Gammon Forum

cd ../area
valgrind --tool=memcheck ../bin/swr 1530

==25819== Conditional jump or move depends on uninitialised value(s)
==25819==    at 0x4E52A3: strip_color (functions.c:353)
==25819==    by 0x4E63B4: is_name2 (handler.c:529)
==25819==    by 0x4E6549: nifty_is_name (handler.c:566)
==25819==    by 0x4EAC4F: get_char_world (handler.c:1777)
==25819==    by 0x455099: find_location (act_wiz.c:750)
==25819==    by 0x473D48: do_goto (build.c:864)
==25819==    by 0x4F6986: interpret (interp.c:418)
==25819==    by 0x49D7AF: game_loop (comm.c:602)
==25819==    by 0x49C6FA: main (comm.c:263)
==25819== 
==25819== Conditional jump or move depends on uninitialised value(s)
==25819==    at 0x4E5268: strip_color (functions.c:355)
==25819==    by 0x4E63B4: is_name2 (handler.c:529)
==25819==    by 0x4E6549: nifty_is_name (handler.c:566)
==25819==    by 0x4EAC4F: get_char_world (handler.c:1777)
==25819==    by 0x455099: find_location (act_wiz.c:750)
==25819==    by 0x473D48: do_goto (build.c:864)
==25819==    by 0x4F6986: interpret (interp.c:418)
==25819==    by 0x49D7AF: game_loop (comm.c:602)
==25819==    by 0x49C6FA: main (comm.c:263)

/*
 * Strips colors from a string --Keberus
 */
char *strip_color( const char *str )
{
   static char newstr[MAX_STRING_LENGTH];
   int i, j;

   if( !str || str[0] == '\0' )
   {
      return "";
   }

   for( i = j = 0; ( str[i] != '\0' ); i++, j++ )  // <--- line 353
   {
      if( str[i] == '&' )   // <---- line 355
      {
         i = i + 2;
      }
      newstr[j] = str[i];
   }
   newstr[j] = '\0';
   return newstr;
}

   int i = 0, j = 0;

#0  strip_color (str=0x7fffffffbc00 "&w") at functions.c:348

& w 0x00

/*
 * Strips colors from a string --Keberus
 */
char *strip_color( const char *str )
{
   static char newstr[MAX_STRING_LENGTH];
   int i, j;

   if( !str || str[0] == '\0' )
   {
      return "";
   }

   for( i = j = 0; str[i]; )
   {
      if( str[i] == '&' )
         i += 2;  // skip ampersand and color code
      else
         newstr[j++] = str[i++];  // copy character
   }
   newstr[j] = '\0';
   return newstr;
}

/*
 * Strips colors from a string --Keberus
 */
char *strip_color( const char *str )
{
   static char newstr[MAX_STRING_LENGTH];
   int i, j;

   if( !str || str[0] == '\0' )
   {
      return "";
   }

   for( i = j = 0; str[i]; )
   {
      if( str[i] == '&' )
        {
        i++;    // skip ampersand
        if (str [i])
          i++;  // skip color code
        }
      else
         newstr[j++] = str[i++];  // copy character
   }
   newstr[j] = '\0';
   return newstr;
}

(gdb) bt
#0  0x00007fff931e182a in __kill ()
#1  0x00007fff960ccb6c in __abort ()
#2  0x00007fff9609e89f in __chk_fail ()
#3  0x00007fff9609e6d9 in __sprintf_chk ()
#4  0x00000001000194f8 in show_char_to_char_0 (victim=0x100941000, ch=0x100943a00) at act_info.c:520
#5  0x000000010001adab in show_char_to_char (list=0x100941000, ch=0x100943a00) at act_info.c:816
#6  0x000000010001bad1 in do_look (ch=0x100943a00, argument=0x1002400d4 "") at act_info.c:1008
#7  0x0000000100047ee3 in transfer_char (ch=0x100941000, victim=0x100943a00, location=0x1004bcfc0) at act_wiz.c:813
#8  0x00000001000481f6 in do_transfer (ch=0x100941000, argument=0x7fff5fbff625 "") at act_wiz.c:864
#9  0x0000000100116d56 in interpret (ch=0x100941000, argument=0x7fff5fbff621 "nick") at interp.c:418
#10 0x00000001000a57dd in game_loop () at comm.c:602
#11 0x00000001000a436c in main (argc=2, argv=0x7fff5fbffb08) at comm.c:263