为什么我的程序给我一个 Abort Trap: 6 错误？

Question

这段代码应该对字符串数组进行排序，但是在选择排序中主循环的第二次迭代时，它给了我一个 Abortion Trap: 6 错误。我在 Mac 的终端上运行它。这是代码

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

int letSize = 20;
int vecSize;
char **array1;

void selectionSort (int low, int high)
{
    char *temp = malloc ((letSize + 1) * sizeof (char));
    int i = 0, j = 0;

    for (i = low; i < high - 1; i++) {
        int indexOfMin = i;
        for (j = i + 1; j < high; j++)
            if (strcmp (array1[j], array1[indexOfMin]) < 0)
                indexOfMin = j;
        //after second main loop, error occurs
        strcpy (temp, array1[i]);
        strcpy (array1[i], array1[indexOfMin]);
        strcpy (array1[indexOfMin], temp);
    }
}

int main ()
{
    int i, j;

    printf ("Enter size of items to be sorted: ");
    scanf ("%d", &vecSize);
    array1 = malloc (vecSize * sizeof (char *));
    for (i = 0; i < vecSize; i++)
        array1[i] = malloc ((letSize + 1) * sizeof (char));

    srand (time (NULL));
    for (i = 0; i < vecSize; i++) {
        for (j = 0; j <= letSize; j++) {
            if (j != letSize) {
                char randLet = 'A' + (random () % 26);
                array1[i][j] = randLet;
            } else
                array1[i][j] = '\0';
        }
    }
    selectionSort (0, vecSize);
}

这是给我带来麻烦的代码。它编译没有任何问题，它也接受用户的输入，但之后它给了我 abort trap 的错误： 6. 可能是什么原因造成的？ 提前致谢。

Answer 1

问题是当j == indexOfMin（或j == i）尝试使用strcpy（您可以使用memmove，而不是strcpy）复制重叠的内存区域时，您尝试进行复制。来自man strcpy

strcpy()函数复制src指向的字符串，包括 终止空字节 ('\0')，指向 dest 指向的缓冲区。 The strings may not overlap, ....

您只需要检查并仅在j != indexOfMin 时进行复制，以防止尝试将字符串复制到自身之上。例如：

void selectionSort (int low, int high)
{
    char *temp = malloc ((letSize + 1) * sizeof (char));
    int i = 0, j = 0;

    for (i = low; i < high - 1; i++) {
        int indexOfMin = i;
        for (j = i + 1; j < high; j++)
            if (strcmp (array1[j], array1[indexOfMin]) < 0)
                if (j != indexOfMin) {
                    indexOfMin = j;
                    strcpy (temp, array1[i]);
                    strcpy (array1[i], array1[indexOfMin]);
                    strcpy (array1[indexOfMin], temp);
                }
    }
    free (temp);
}

还记得free (temp)，否则你肯定会出现内存泄漏。