c - 当我将指针设置为 NULL 时我的程序崩溃

Question

这让我抓狂，我已经完成了作业，但出于某种原因

for (int i = 0; i < room->num_of_challenges; i++) { // need a check
        if (strcmp(room_to_enter, room->challenges[i].challenge->name) == 0) {
            room->challenges[i].start_time = start_time;
            room->challenges[i].visitor = malloc(sizeof(room->challenges[i].visitor));
            room->challenges[i].visitor->current_challenge = malloc(sizeof(room->challenges[i].visitor->current_challenge));
            *room->challenges[i].visitor->current_challenge = room->challenges[i];
            *room->challenges[i].visitor->room_name = NULL;
            *room->challenges[i].visitor->room_name = malloc(strlen(room_to_enter)+1);
            strcpy(*room->challenges[i].visitor->room_name, room_to_enter);
            inc_num_visits(room->challenges[i].challenge);
        }
    }

此时程序由于某种原因崩溃了:

*room->challenges[i].visitor->room_name = malloc(strlen(room_to_enter)+1);

这是一个 3 页的代码，每页都有标题，每页大约 300 行，所以我不能全部发布，这里还有结构:

struct SChallengeActivity;
typedef struct SVisitor
{
  char *visitor_name;
  int visitor_id;
  char **room_name;
  struct SChallengeActivity *current_challenge;
} Visitor;


typedef struct SChallengeActivity
{
   Challenge *challenge;
   Visitor *visitor;
   int start_time;
} ChallengeActivity;


typedef struct SChallengeRoom
{
   char *name;
   int num_of_challenges;
   ChallengeActivity *challenges;
} ChallengeRoom;

我们不能编辑这个结构，因为它是由作业给出的，问题是我试图将这一行设置为 NULL，如下所示:

 *room->challenges[i].visitor->room_name = NULL;

它仍然在该行崩溃，由于某种原因它无法到达结构中的那个字段。 注意:忽略编码，我知道我应该在每一行之后检查 malloc，但现在我想让它工作，我整夜都在让它工作，但我做不到，有什么帮助吗？ 谢谢

Answer 1

为访问者分配内存时，您只是在分配指针的大小。挑战中的访客字段只是指向访客的指针。因此，您没有分配足够的内存。你应该使用:

malloc(sizeof(Visitor))

获取访问者结构的真实大小并分配足够的空间。

此外，在分配之前将指针设置为空是无用的。对 malloc() 的调用将覆盖它们。