我正在制作一个 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
,缓冲区溢出问题得到解决.感谢推荐阅读!
最佳答案
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]
处停止读取.
关于c - 内存重新分配期间缓冲区溢出,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/41643109/