Question

我正在尝试使用realloc来动态创建结构的实例，并在我去的时候用临时结构中的数据填充它。当程序第二次到达malloc结构的指针时程序崩溃，但我不知道应该如何构造这个函数。我有以下代码：

#define MAX_STRING 50

struct data {
int ref; 
int port;
char data[MAX_STRING+1];
}valid, invalid;

void read_file(FILE *file);
void validate(struct data* temp); 

int g = 0;

int main(){

    char inputfile[100];
    FILE *file = fopen("file.txt" , "r");

    if (file != NULL){
       read_file (file);
    }

    else{
    // Some code here..
    }

    return 0;
}  

void read_file(FILE *file){

    struct data* temp = malloc(sizeof(struct data));

    char buf[1024];
    while(!feof(file)){

       fgets(buf, sizeof buf, file))

       sscanf(buffer, "%d.%d.%s", &temp->ref, &temp->port,  &temp->data);

       validate(temp);
       g++;

    }
}

void validate(struct data* temp){

    if((some condition) && (some condition))
    {
        create_valid(temp);
    }

    if((some condition) && (some condition))
    {
        create_invalid(temp);
    }
}

我不确定如何构建以下函数：

int create_vaild(struct data* temp){

    struct data* valid = malloc(sizeof(struct data)); <<<<<<<<< Line that crashes 

    valid = realloc(valid, g * sizeof(struct data));

    valid[g] = *temp;

    if (valid[g] == NULL){
        //error.
    };
    printf("\n%i:%i:%s\n", (valid+g)->ref, (valid+g)->port, (valid+g)->data);



return 0;

}

Answer 1

我看到一个潜在的问题：

你有g设置为0，即

int g =0;

在调用create_valid()之前，您没有递增它。您正在使用此值在该函数内分配内存：

valid = realloc(valid, g * sizeof(struct data));

所以现在g是0。

稍后在下一行中取消引用此指针

valid[g] =  *temp;

这是你没有分配的一些内存，因为realloc()没有为你分配内存，因为你将0传递给它。防止崩溃。

在动态结构数组中使用realloc

1 个答案: