0

我有这两个文件：

测试.h

#ifndef TEST_H
#define TEST_H

typedef struct _vertex_node
{
  double x;
  double y;
  struct _vertex_node *next;
} vertex_node;

static void add_vertex(vertex_node **node, const double x, const double y);
static void dty_vertex(vertex_node **node);

#endif // TEST_H

测试.c

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

#include "test.h"


static void add_vertex(vertex_node **node, const double x, const double y)
{
  if(NULL == (*node))
  {
    (*node) = malloc(sizeof(vertex_node));
    (*node)->x = x;
    (*node)->y = y;
    (*node)->next = NULL;
  }
  else
    add_vertex(&((*node)->next), x, y);
}

static void dty_vertex(vertex_node **node)
{
  if(NULL != (*node)->next)
    dty_vertex(&((*node)->next));

  free(*node);
}

int main(int argc, char **argv)
{
  vertex_node *node;
  vertex_node *iterator;

  add_vertex(&node, 0, 0);
  add_vertex(&node, 1, 0);
  add_vertex(&node, 1, 1);
  add_vertex(&node, 0, 1);

  iterator = node;

  while(NULL != iterator)
  {
    printf("x: %f, y: %f\n", iterator->x, iterator->y);

    iterator = iterator->next;
  }

  dty_vertex(&node);

  return 0;
}

我正在使用gcc -Wall -ggdb test.c -o test命令来编译它。

当我尝试运行它时，它在释放内存时给我一个分段错误，这里有什么问题？

4

5 回答 5

3

您需要初始化node：

vertex_node *node = NULL;

如果你不这样初始化，检查：

if(NULL == (*node))
  { /* */ }

第一次调用将失败，这部分被执行：

add_vertex(&((*node)->next), x, y);

中的任意值node。此时您可能会遇到 seg-fault，或者当您尝试释放节点时（如果您不走运）。

于 2012-08-14T15:27:26.330 回答
3

堆栈上的变量不是零初始化的。尝试以下操作：

vertex_node *node = NULL;
于 2012-08-14T15:27:29.263 回答
1

顶点节点应显式设置为 NULL。您得到的错误可能是由于释放了非分配内存。

于 2012-08-14T15:29:24.877 回答
0

您正在释放非堆内存。函数退出时释放堆栈内存。

于 2012-08-14T15:29:06.483 回答
0

您必须检查*node自己是否NULL在dty_vertex.

于 2012-08-14T15:27:19.840 回答