函数结束时的 C++ 段错误。参考线 = 右大括号

Question

总结：非常感谢大家！下面发布的所有回复都是正确的。最初的错误是我忘记为 NULL 终止符留出空间。strcpy() 是一个危险的函数，因为当我使用它时，它不知道“字符串”的结尾是什么时候。因此，strcpy() 抓取了大量数据并覆盖了返回地址。

编辑：从程序中添加了更多代码

解决：老实说，我最初的实现是垃圾。如果我想换出数组的元素，我什至不知道为什么要这样写交换。（当时，每个元素中只有一个字符数组。所以我能够摆脱旧的实现）。我已经把它改写为：

void swap(ArrayElement list[], int index1, int index2) {
     ArrayElement temp;
     temp = list[index1];
     list[index1] = list[index2];
     list[index2] = temp;
}

---

我在以下函数结束时遇到分段错误问题。

struct ArrayElement {
    char data[SIZE_OF_ELEMENT];
    // Implemented this way so that I can expand to multiple values later on
}

//In main:
ArrayElement* list = new ArrayElement[NUM_OF_ELEMENTS];

void swap(ArrayElement list[], int index1, int index2) {
     char temp[SIZE_OF_ELEMENT];
     strcpy(temp, list[index2].data);
     strcpy(list[index2].data, list[index1].data);
     strcpy(list[index1].data, temp);
}

错误是第 45 行的分段错误，它是函数的结尾大括号。这是使用 g++ 编译的。我使用 gbd 尝试调试它，一切正常，直到它碰到花括号。

如果需要，我可以从程序中提供更多代码。我不想发布整个内容，因为这是一个课程。

Answer 1

我最好的猜测是，字符串 atlist[index2].data大于temp[]并通过复制覆盖了堆栈和返回地址。

尝试插入长度测试：

#include <iostream>

...
int n = strlen(list[index2].data);
std::cerr << "len=" << n << ", SIZE_OF_ELEMENT=" << SIZE_OF_ELEMENT << std::endl;

看看，如果n(list[index2].data) 大于SIZE_OF_ELEMENT