Realloc()行为

Question

我试图将通过argv[]传递到c-字符串中的多个参数连接起来。我不使用char数组，因为我不希望对通过argv[]传递的字符数量有任何限制。

这是我如何执行程序的一个例子。

可执行-cmd rotn 13 Hello

char *externalCommand = NULL;
    if (argc >= 3 && strcmp(argv[1], "-cmd") == 0)
        {
            externalCommand = (char *)malloc(sizeof(char) * (strlen(argv[2]) + 1));
            strcpy(externalCommand, argv[2]);
            for (int i = 3; i < argc; ++i)
            {
                int textLength = strlen(externalCommand);
                externalCommand = (char *)realloc(externalCommand, sizeof(char) * (textLength + 1));
                strcpy(externalCommand + textLength, " ");
                strcpy(externalCommand + textLength + 1, argv[i]);
            }
        }

我的目标是以字符串" externalCommand“结束为相等："rotn 13 Hello”当我第一次遍历for循环时，externalCommand是"rotn 13"，这是正常的。

当我第二次遍历循环时，当我运行realloc()时，我会得到奇怪的行为。externalCommand变成了"rotn _G_~_\“。我知道这些是垃圾值，但我不明白为什么realloc()覆盖字符串中的数字"13“。

为了解决这个问题，我可以取代：

externalCommand = (char *)realloc(externalCommand, sizeof(char) * (textLength + 1));

有：

externalCommand = (char *)realloc(externalCommand, sizeof(char) * (textLength + 3));

它不会覆盖值13。

我想了解为什么在重新分配时我需要将3添加到textLength中。

Answer 1

您的realloc()没有分配足够的内存来保存要复制到缓冲区中的数据。用这个代替：

externalCommand = (char *)realloc(externalCommand, sizeof(char) * (textLength + 1 + strlen(argv[i]) + 1));

也就是说，如果预先计算缓冲区大小，则可以去掉realloc()并使用单个malloc()。

char *externalCommand = NULL;
if (argc >= 3 && strcmp(argv[1], "-cmd") == 0)
{
    int textlength = strlen(argv[2]);
    for (int i = 3; i < argc; ++i)
        textLength += (1 + strlen(argv[i]));

    externalCommand = (char *)malloc(sizeof(char) * (textLength + 1));

    strcpy(externalCommand, argv[2]);
    for (int i = 3; i < argc; ++i)
    {
        strcat(externalCommand, " ");
        strcat(externalCommand, argv[i]);
    }
}

Answer 2

realloc()将ptr指向的内存块的大小更改为作为输入传递的字节大小。

您缺少argvi的分配。

您应该将所需的总大小作为realloc的参数。

在realloc中，您需要给出大小为textLength + strlen(argvi) + 2