c - 内存重新分配期间缓冲区溢出

Question

我正在制作一个 C 程序，将 12 小时制转换为 24 小时制，输入格式为 HH:MM:SSAM 或 HH:MM:SSPM，24 小时制输出格式为 HH:MM:SS

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

int main(){
    char *time = malloc(11 * sizeof(char));
    scanf("%11s", time);
    if (time[8] == 'A' || time[8] == 'P')
    {

        if (time[8] == 'A')
        {
            time = realloc(time, (9 * sizeof(char)));                
            printf("%s\n", time);
        }
        else
        {
            time = realloc(time, (8 * sizeof(char)));                
            char str[3];
            sprintf(str, "%c%c", time[0], time[1]);
            int hours;
            hours = atoi(str);
            int milhours;
            milhours = hours + 12;
            char milstr[3];
            sprintf(milstr, "%d", milhours);
            time[0] = milstr[0];
            time[1] = milstr[1];
            printf("%s\n", time);
        }   
    }
    else
    {
        printf("give a standard format\n");
        return 0;
    }
    return 0;
}

没有编译错误，但是程序因为缓冲区溢出而无法运行。当我减小动态数组 time 的大小时，是否有必要删除 time 的最后 2 个元素？

编辑:我为 NULL 终止符更新了 time、str 和 milstr，缓冲区溢出问题得到解决.感谢推荐阅读!

Answer 1

EDIT: I updated time, str and milstr for NULL terminator and the buffer overflow problem is resolved. Thanks for the recommended reading!

你还有问题。

char *time = malloc(11 * sizeof(char));
scanf("%11s", time);

那个malloc分配 11 个字节，但是 scanf需要 12。这是因为 C 字符串以空字节结尾，您总是需要再分配一个字节。

---

至于您所做的所有重新分配，您似乎正在这样做以截断字符串。这对于 3 个字节来说太过分了，而且它也不会被截断。因为realloc可能会返回相同的指针，你不能指望收缩会将后面的内存归零并截断字符串。当它返回不同的指针或增加内存时，您不能指望它。仅适用于 calloc你能确定分配的内存已归零吗？

相反，跳过 realloc并插入一个空字节以截断字符串。

if (time[8] == 'A')
{
    time[8] = '\0';
    puts(time);
}

time仍然是 12 个字节(一旦 malloc 已修复)，但空字节告诉 C 在 time[8] 处停止读取.