【问题标题】:Understanding results with valgrind用 valgrind 理解结果
【发布时间】:2020-12-01 06:35:54
【问题描述】:

对于以下代码,我有以下定义:

typedef struct string {char* data; int length;} string;

如果我使用 valgrind 运行以下代码,我得到条件跳转或移动取决于未初始化的值和 seg。故障:

string* s = (string*) malloc(sizeof(string));
strcpy("Hello", s->data);
free(s);

首先,我不明白为什么会出现上述错误。 我想如果我添加到该代码free(s->data) 它会释放内存但程序会运行正常。

我的看法: 我知道sizeof(string) 等于 4(指向 char 的指针)+ 4(int)= 8。 然后我们为 s 分配 8 位。 strcpy 会将字符串复制到数据中,但我在这里遇到了问题。为什么?

【问题讨论】:

  • “1(指向字符的指针)”?现代 PC 上的典型指针大小为 4 或 8。您的环境是什么?另外我猜分配应该是 5 个字节,而不是 5 位。
  • strcpy("Hello", s->data);?你确定?
  • @MikeCAT,你说得对。它是 4 位。然后我有 8 位
  • First of all, I can't understand why I got above error.. 从阅读man pages开始
  • 通过malloc() 分配的缓冲区未初始化。 s->data 也未初始化。

标签: c pointers memory-leaks valgrind dynamic-memory-allocation


【解决方案1】:

有多个问题:

string* s = (string*) malloc(sizeof(string));

应该更好

string* s = malloc(sizeof(*s));

s->data 分配内存,但不使s->data 指向任何有效的内存位置。如果你想使用内存位置,你需要确保它指向一个有效的内存位置。例如:您需要分别为s->data malloc()

也就是说,syntax for strcpy() 说,它是 strcpy(dest, source),所以在你的情况下

strcpy("Hello", s->data);

尝试

  • 从统一的内存位置读取
  • 写入字符串文字

其中任何一个都会调用undefined behaviour

你应该写

strcpy(s->data, "Hello");

在确保s->data 是有效目的地之后。

【讨论】:

  • s->data 仍未初始化。
  • string* s = (string*) malloc(sizeof(string)); 是正确的,但是再添加:
  • s->data= maloc(sizeof("hello"));
  • @PaulOgilvie 当然是第一种方法,但是考虑到未知的用法,我们可能需要更多内存 - 只是说。
猜你喜欢
  • 2014-10-06
  • 2011-12-03
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2017-03-11
  • 1970-01-01
  • 1970-01-01
  • 2023-03-25
相关资源
最近更新 更多