Original Post
Well I was writing some word list code. It works OK, everything is perfect, but maybe I do miss some memory leaks, it is wery hard to find them .So il ask your help. Can anyone suggest is there a way to improve my code, or are there some mistakes ? i do have some concerns where i pass new struct to temp pointer. tnx in advance.
#include <stdio.h>
struct WSTRUCT;
//this is our words
struct WSTRUCT *words;
void AddWord(char* word,int length);
void FreeWords();
struct WSTRUCT
{
//word
char* word;
//pointer to next struct NULL if none
struct WSTRUCT *next;
};
//frees memory used
void FreeWords()
{
while(words != NULL)
{
if(words->word != NULL)
free(words->word);
words = words->next;
}
}
void AddWord(char* word,int length)
{
struct WSTRUCT *temp;
temp = words;
//this indicates that the are no
//words in a list
//so we add word and exit
if(words == NULL)
{
//now we create new structure
words = (struct WSTRUCT*)malloc(sizeof(struct WSTRUCT));
//we create a new word
words->word = (char*)malloc(sizeof(char) * length);
//and write a value to it
sprintf(words->word,"%s",word);
//and point the next pointer
//to NULL
words->next = NULL;
return;
}
else
while(temp->next != NULL)
{
temp = temp->next;
}
//now temp is the last structure we have
//so let's make it work
temp->next = (struct WSTRUCT*)malloc(sizeof(struct WSTRUCT));
//we create a new word
temp->next->word = (char*)malloc(sizeof(char) * length);
//and write a value to it
sprintf(temp->next->word,"%s",word);
//and point the next pointer
//to NULL
temp->next->next = NULL;
return;
}
int main()
{
//add new word
AddWord("word1",6);
AddWord("word2",6);
printf("ALLOK\n");
//frees memory
FreeWords();
return 0;
}
