c - EXC_BAD_ACCESS(代码=1，地址=0x0)

Question

我试图从全局变量源填充结构的链接列表，但在 strcpy() 行得到 BAD_ACCESS。使用C。想知道是否有人可以指出这个问题。

全局结构声明如下:

#include "data_structs.h"
#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define LINE_LEN   256
#define NUL        '\0'

table_entry_t reg_list[]=
{
{"R0",0},{"PC",0},{"R1",1},{"SP",1},{"R2",2},{"SR",2},{"R3",3},
{"R4",4},{"R5",5},{"R6",6},{"R7",7},{"R8",8},{"R9",9},{"R10",10},
{"R11",11},{"R12",12},{"R13",13},{"R14",14},{"R15",15},{"R16",16}
};

...这些结构如下所示(如下)，并在 .h 文件中定义。

typedef struct
{
    char label[20];
    int address;

}table_entry_t;

typedef struct
{
    table_entry_t *data;
    void *next;
} List_node_t;

typedef struct
{
    List_node_t *head;
}list_t;

链表使用以下方式初始化:(如下)。 EXC_BAD_ACCESS 出现在“strcpy(new_node->data->label,reg_list[i].label);”行

boolean List_init (list_t *list)
{
int all_ok = False;
int i=0;
char* temp[3];

if (list != NULL) {
    list->head = NULL;

    //Add Register Labels

    while(i<20)   // 20 register labels
    {
        List_node_t *new_node = NULL;    
        new_node = (List_node_t *) malloc( sizeof( List_node_t));
        strcpy(new_node->data->label,reg_list[i].label); <---BAD ACCESS
        new_node->data->address = reg_list[i].address; 
        new_node->next = list->head->next;
        list->head->next = new_node;
    }
    all_ok = True;
}
return all_ok;
}

欣赏新鲜的眼睛。问候。

Answer 1

风格注释:C 编程世界的其余部分通常使用 list_node_t 或 ListNode(分别为蛇形表示法和驼峰命名法)。您的符号有点令人困惑。

您正在分配List_node_t，但其中的data指针未初始化。你很幸运，它是 NULL，你知道有问题。它也可能是程序内的随机地址，此后任何程序行为都是可接受的，包括您的程序是否显示玩俄罗斯方 block 的 3D 窗口。

您也可以分配它，但这很可能不是您想要做的。更好的方法是将 List_node_t 的定义更改为:

typedef struct List_node_t
{
    table_entry_t data;
    struct List_node_t *next;
} List_node_t;

我做了两项更改。第一个也是重要的一个是我将数据更改为内联，因此一次分配会同时分配数据和节点。显然，您还需要调整代码其余部分中访问它的语法。

第二个也有点风格，但我强烈推荐它。应正确键入指向下一个的指针。您无法从结构定义中访问 typedef，但可以通过名称访问结构本身。为此，我为该结构体指定了一个名称(与 typedef 相同)，并使用它来键入 next 指针。