Well I personally haven't touched that part of mine I don't think :D but I don't think I was suggesting it be freed before the clan_lookup, I was suggesting doing something like michael's idea where you declare:
char *temp; int clan; temp = fread_string(fp); clan = clan_lookup(temp); free_string(temp); Richard Lindsey -----Original Message----- From: Christopher Bunting [mailto:[EMAIL PROTECTED] Sent: Thursday, April 07, 2005 11:27 AM To: Richard Lindsey Cc: [email protected] Subject: Re: Memory issues. Hello, Getting back to the post and to that block of code.. Out of curiosity, How are you freeing the string for name before it's being used for the clan lookup? To my knowledge, you free the string after the function is complete. Not before it needs to do something else. Chris

