c - 为什么 strcpy 不能正常工作

Question

这个问题已经有答案了:

Crash or "segmentation fault" when data is copied/scanned/read to an uninitialized pointer (5 个回答)

已关闭 5 年前。

我想将字符串数组的每个元素复制到另一个字符串数组，但是当我复制一个字符串时，整个数组都会更改为该字符串。以下是代码集。

这是有问题的代码行:

strcpy(MsgList[i].ga_data, a_Database[i].ga_data);

当我检查内容时a_Database[i].ga_data，如下

"1240,message 7:War of the worlds"

"1238,message 5:Life of this world"

"1236,message 3:world is not enough"

"1235,message 2:What a world!"

因此，不要在MsgList[i].ga_data中填充相同的内容，结果只是“1235，消息2:多么美好的世界啊!” 4次，这是最后复制的元素。

typedef enum
{
  ACK,
  NACK,
  DELETE
}eMsgStatus_t;

typedef struct _Message
{
  eMsgStatus_t status;
  char *ga_data;
  uint16_t time;
}Message_t;
Message_t MessageList[8]={
  {ACK,"1234,message 1:Hello world",1000},
  {NACK,"1235,message 2:What a world!",1011},
  {NACK,"1236,message 3:world is not enough",1022},
  {ACK,"1237,messsge 4:Cruel world",1033},
  {NACK,"1238,message 5:Life of this world",1044},
  {ACK,"1239,message 6:Around the world in 80 days",1055},
  {NACK,"1240,message 7:War of the worlds",1066},
  {ACK,"1241,message 8:End of World",1077}
}; 

Message_t a_Database[20];
Message_t MsgList[20]= {0};

int main()
{
  for (i = 0; i < idx /* total unread message */; ++i)
  {
      strcpy(MsgList[i].ga_data, a_Database[i].ga_data);
      MsgList[i].ga_data[14] = '\0';
  }
}

uint8_t  GetMessages (Message_t *pg_Message)
{
  char i;uint8_t idx = 0;
   for (i = 0; i < 8 /* total message */; ++i)
  {
    if(NACK == MessageList[i].status)
    {
      pg_Message[idx].status = MessageList[i].status;
      pg_Message[idx].ga_data = MessageList[i].ga_data;
      pg_Message[idx].time = MessageList[i].time;
      idx++;
    }
  }
  return idx;
}

Answer 1

首先，你在这里破坏了内存:

strcpy(MsgList[i].ga_data, a_Database[i].ga_data);

因为我看不到 MsgList[i].ga_data 的分配是在哪里完成的，我认为不是。

然后在此处通过魔数(Magic Number)“14”制作以 null 结尾的字符串:

 MsgList[i].ga_data[14] = '\0';

也可能导致一些内存损坏。

甚至假设您在某处进行分配，您可以在 MsgList[i].ga_data 中观察到完整字符串，这很奇怪，因为您在 14 处进行了 null 终止，它应该类似于:

1235,message 2                                                            
1236,message 3
1238,message 5 
1240,message 7