2

我在 C 中使用链表时遇到问题，我只在 C++ 中完成了诸如此类的数据结构。

Gdb 给了我一个

Program received signal SIGSEGV, Segmentation fault. 0x0804a23c in addArg (base=0x1, argument=0x804e410 "is") at myshell.c:42 42 while ( (curr != NULL) && (curr->n != NULL) )

我熟悉与内存有关的分段错误，但是我认为我已经正确分配了内存。我究竟做错了什么？

addArg 被称为addArg(currentCmd->args, lexeme);并且 currentCmd 是指向节点结构的指针

struct lnode {
 char *x;
 struct lnode *n;
};


struct node
  {
    char *command;
    struct lnode *args;
    int input;
    int output;
    int error;
    char *in;
    char *out;
    char *err;
    struct node *next;
    struct node *prev;
  };



void addArg(struct lnode *base, char *argument)
 {
 struct lnode *curr = base;

//this is line 42
  while ( (curr != NULL) && (curr->n != NULL) )
    curr = curr->n;

   curr -> n = malloc(sizeof(struct lnode));
   curr = curr->n;
   curr->x = strdup(argument);
   curr->n = NULL;
 }




struct node* createNode(char *command_, int input_, int output_, int error_, char *in_, char *out_, char *err_, struct node *prev_)
  {
  struct node *n;
  n = malloc(sizeof (struct node));
  n->command = strdup(command_);
  n->prev = prev_;
  n->next = NULL;
  n->input = input_;
  n->output = output_;
  n->error = error_;
  n->in = in_;
  n->out = out_;
  n->err = err_;
  n->args=malloc(sizeof(struct lnode));

 return n;
  }
4

3 回答 3

0

我通过对内存管理进行必要的更改并lnode *args 解决了这个问题。lnode args

于 2012-11-17T22:11:12.600 回答
0

看起来currentCmd->args是一个无效的指针。也许是指向free()d 内存的指针。或未初始化的指针，或指向超出范围的局部变量的指针（尽管后两者在这里似乎不是这种情况）。

或者，您可能不小心在程序的其他地方覆盖了越界内存。指针问题并不总是在失败点上。有时它们在较早的代码中，甚至是不相关的代码。

于 2012-11-17T21:21:39.540 回答
0

我从您的 gdb 输出中可以看到，问题while ( (curr != NULL) && (curr->n != NULL) )是如果 curr == NULL 您仍在尝试访问 curr->n 进行比较，因此您应该将该条件更改为仅比较 curr，并仅处理 curr->n如果 curr 不为空，如果 curr->n == NULL，可能会立即打破 cicle。

于 2012-11-18T17:37:55.487 回答