c - 为什么这段代码的行为不可预测？

Question

我正在开发一款主机游戏，其中包含以下代码:

typedef struct player{
        char *name;
        /* ... */
        char location;
        char traveltime;
        /* ... */
}pl;

typedef struct planet{
        char *name;
        /* ... */
}planet;

pl *players;
planet plan[22];

pl *players 已分配

players=malloc(NPLAYERS*sizeof(pl));

其中 NPLAYERS 是玩家数量。plan[] 是游戏中所有行星的数组。

players[i].location

如果players[i].traveltime==0，

是作为plan[]的下标的玩家的位置。如果players[i].traveltime>0，则玩家正在前往players[i].location。因此，当玩家旅行时，我想显示一个 ncurses 窗口，说明“正在前往(行星)的途中”。

为此我使用:

char *tmp, msg[]="PLAYER 1", i;
for(i=0; i!=NPLAYERS; ++i){
            infobox(msg);
            if( players[i].traveltime>0){
                    tmp=malloc( sizeof("en route to ")+sizeof(plan[ players[i].location ].name)+4)
                    strcpy(tmp, "en route to ");
                    strcat(tmp, plan[ players[i].location ].name);
                    strcat(tmp, "..\0");
                    infobox(tmp);
                    free(tmp);
            }
            ++msg[7];
  }

其中 infobox(char msg[]) 将包含消息的 ncurses 窗口打印到 stdout，NPLAYERS 是玩家数量。这个想法是，这段代码循环遍历所有玩家，检查他们是否正在旅行，如果是，则打印一条消息，说明他们的目的地。这十次中有九次有效，但有时它会在 free(tmp) 处出现段错误，在 malloc 处出现段错误或者打印

    ***** glibc detected *** ./st: malloc(): memory corruption [a hex number] ***

在 malloc 之后。为什么会这样？我该如何解决？

了解我在一台两年前的笔记本电脑上使用 Arch Linux 可能会有所帮助。

Answer 1

在sizeof(plan[players[i].location ].name)中，您正在获取指针的大小。您可能想要最大行星名称的大小......

    tmp=malloc( sizeof("en route to ")+sizeof(plan[ players[i].location ].name)+4);

对比

    //tmp = malloc(13 + sizeof(char *) + 4);
    //tmp = malloc(17 + sizeof(char *));

    tmp = malloc(1700); /* arbitrary big enough value :) */