使用stringstreams进行strdup会导致valgrind错误

时间:2012-07-18 08:37:37

标签: c++ valgrind

我有以下问题:我正在编写一个必须包装C库的C ++程序,因此当我与库交互时,我总是必须使用char*代替std::string操作。为了避免尽可能地使用char*,我使用stringstreams进行格式化,例如:

#include <iostream>
#include <sstream>
#include <string.h>
#include <cstdlib>

using namespace std;

int main(int argc, char** argv)
{
  ostringstream str;

  str << argv[0] << "+" << "hello";

  const char *s = str.str().c_str();

  char *y = strdup(s);

  // this I would give to a library function

  cout << y << endl;

  free(y);

  return 0;
}

就输出而言,程序正确输出“./test+hello”。但是,valgrind给了我很多类型

的错误
==30350== Invalid read of size 1
==30350==    at 0x402B858: __GI_strlen (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==30350==    by 0x4213475: strdup (in /usr/lib/libc-2.16.so)
==30350==    by 0x41B2604: (below main) (in /usr/lib/libc-2.16.so)
==30350==  Address 0x4341274 is 12 bytes inside a block of size 25 free'd
==30350==    at 0x4029F8C: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==30350==    by 0x410387A: std::string::_Rep::_M_destroy(std::allocator<char> const&) (in /usr/lib/libstdc++.so.6.0.17)
==30350==    by 0x41B2604: (below main) (in /usr/lib/libc-2.16.so)

我做错了什么?

1 个答案:

答案 0 :(得分:6)

 const char *s = str.str().c_str();

str()返回一个字符串对象。您使用c_str获得指向它的某些内部数据的指针,然后在该行的末尾删除字符串对象。但是你仍然有一个指向已删除内部字符串的指针。

你需要这样做 -

std::string s = str.str();
const char* s = s.c_str()

确保不删除字符串。