Solved

free nodes in a link list

Posted on 1998-08-14
5
179 Views
Last Modified: 2010-04-15
I am writting an appan I have to read info from a database so I am going to the info in a link list and right now I am just building my link list functions...well I can add info to the link list...but when it comes to free all the nodes in the link list when ever I go to free the "main_tail" node my program crashes...anyone have clue as to why this is happening??  The code I am using is below

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <malloc.h>

struct file_collection
{
      char *filename;
      char *dir;
      struct file_collection *prev, *next;
};

void addrec(struct file_collection **, struct file_collection**, char *, char *);
void allocspace(struct file_collection **, struct file_collection **, struct file_collection **);
void freeallnodes(struct file_collection **, struct file_collection **);

void main(void)
{
      struct file_collection *main_top, *main_tail;

      main_top=main_tail=NULL;
      addrec(&main_top, &main_tail, "this is a test", "this is a dir test");
      addrec(&main_top, &main_tail, "this is a test2", "this is a dir test2");
      freeallnodes(&main_top, &main_tail);
}


void addrec(struct file_collection **main_top, struct file_collection **main_tail, char *temp_filename, char *temp_dir)
{
      struct file_collection *curr, *top, *tail;
                 
                 top = *main_top;
      tail = *main_tail;

      allocspace(&top, &curr, &tail);
                  curr->filename = temp_filename;
      curr->dir = temp_dir;

      *main_top = top;
      *main_tail = tail;
}

void allocspace(struct file_collection **top, struct file_collection **curr, struct file_collection **tail)
{
      *curr = (struct file_collection *)malloc(sizeof(struct file_collection));
      (*curr)->prev=(*curr)->next=NULL;
      if (*top == NULL)
      {
            *top = *curr;
            *tail = *curr;
      }
      else
      {
            (*curr)->prev = *tail;
            (*tail)->next = *curr;
            (*tail) = (*curr);
      }
}

void freeallnodes(struct file_collection **main_top, struct file_collection **main_tail)
{
      struct file_collection *p, *curr;

      for(p=(*main_top);p != NULL; p=p->next)
      {
            curr = p->next;
            if (curr != NULL)
                  free(curr);
      }

      if (*main_top != NULL)
            free(*main_top);
      if (*main_tail != NULL)
            free(*main_tail);
}
0
Comment
Question by:meetze
  • 3
  • 2
5 Comments
 
LVL 4

Accepted Solution

by:
emmons earned 50 total points
ID: 1252117
FreeAllNodes is releasing a piece of memory, and then dereferencing it in the for loop. You need to grab the "next" pointer before you free up the "curr" pointer.
0
 
LVL 4

Expert Comment

by:emmons
ID: 1252118
Try something like:
for( p=(*main_top); p!= NULL; p=curr) {
      curr = p->next;
      if ( p != NULL)
            free( p);
}


0
 

Author Comment

by:meetze
ID: 1252119
I did it like this

void freeallnodes(struct file_collection **main_top, struct file_collection **main_tail)
{
      struct file_collection *p, *curr;
      
      for(p=(*main_top);p != NULL;)
      {
            curr = p;
            if (curr != NULL)
            {
                  p=p->next;
                  free(curr);
            }
            
      }

      if (*main_top != NULL)
            free(*main_top);
      if (*main_tail != NULL)
            free(*main_tail);
}

but now even though my main_top is freed and I am checking to see if it is NULL it still falls through the statement...any reason for this??
0
 

Author Comment

by:meetze
ID: 1252120
well i noticed that I do not need to free main_top or main_tail...thanks for the help..
0
 

Author Comment

by:meetze
ID: 1252121
well i noticed that I do not need to free main_top or main_tail...thanks for the help..
0

Featured Post

ScreenConnect 6.0 Free Trial

At ScreenConnect, partner feedback doesn't fall on deaf ears. We collected partner suggestions off of their virtual wish list and transformed them into one game-changing release: ScreenConnect 6.0. Explore all of the extras and enhancements for yourself!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Summary: This tutorial covers some basics of pointer, pointer arithmetic and function pointer. What is a pointer: A pointer is a variable which holds an address. This address might be address of another variable/address of devices/address of fu…
This is a short and sweet, but (hopefully) to the point article. There seems to be some fundamental misunderstanding about the function prototype for the "main" function in C and C++, more specifically what type this function should return. I see so…
The goal of this video is to provide viewers with basic examples to understand opening and writing to files in the C programming language.
The goal of this video is to provide viewers with basic examples to understand and use conditional statements in the C programming language.

895 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

16 Experts available now in Live!

Get 1:1 Help Now