malloc缓冲区溢出（C）

Question

抱歉，如果您之前未曾回答过此问题，但我一直在疯狂地寻找答案。 我一直在使用malloc创建一个数组：

char *message = (char*) malloc(31 *sizeof(char));

要使用户输入最多30个字符的消息：

 fgets(message, 30, stdin);

然后，在另一个函数中，我尝试将消息转换为小写形式并将其存储在另一个字符串中（希望与消息长度相等）。

int length_message = strlen(message) - 1;

char *temp1 = (char*) malloc(length_message * sizeof(char));

for(int i=0; message[i] != '\0'; i++) 
{
        temp1[i] = tolower(message[i]);     
}

但是当我来显示新字符串时

puts(temp1);

它显示我的输入消息，然后在新行上显示全部乱码。 我不明白这是从哪里来的。

此外，在检查temp1数组的长度和为长度消息创建的值后，请使用：

printf("length of temp array: %d", strlen(temp1));
printf("length of message: %d\n", length_message);

哪个返回了所需的length_message（用户在消息中输入的字符数），但是对于我的临时数组的长度却输入了错误的数字！ 我是否为我的临时阵列错误地分配了内存，或者我没有正确写入该阵列？

从我在其他线程上阅读的内容来看，我似乎有一个缓冲区溢出错误？ 我该如何检查？

提前致谢！

Answer 1

这个：

strlen(message) - 1

当然应该是这样的：

strlen(message) + 1

你打错字了。

Answer 2

您的代码有两个问题。

实际错误是由于您没有将终止0字符从源字符串复制到目标这一事实引起的。 您需要写一个temp[i] = 0; 在for循环之后。

另外，确实确实存在缓冲区溢出的可能性，因为malloc()复制的字符比字符串的长度少一个。 （ 为什么......？你应该分配一个更多 ...）

进一步的问题包括void *的可怕 转换 ，冗余（因而不安全）的sizeof(char)以及硬编码的缓冲区大小。

代替

char *message = (char*) malloc(31 *sizeof(char));

尝试

#define BUF_SIZE 31

char *message = malloc(BUF_SIZE);
fgets(message, BUF_SIZE, stdin);

或者，如果您觉得书呆子，

char *message = malloc(BUF_SIZE * sizeof message[0]);

但在malloc()分配内存时， 切勿在sizeof()使用硬编码类型。 但是无论如何，我认为您根本不需要malloc() 。 为什么不使用自动数组（虽然不是正确的名称，却使用“堆栈分配的”数组）呢？ 然后，您可以在获取用户输入时利用sizeof运算符。

此外， strlen()和其他几个stdlib函数返回size_t 。 您绝对应该使用该类型而不是int 。