2014-08-29 62 views
1

我遇到了一个问题,当我有一个std ::向量在一个结构中,我堆分配该结构,当结构变得free'd,向量泄漏。有关如何防止这种情况的任何想法?内存泄漏与分配的结构包含一个向量

下面的代码:

#include <cstdlib> 
#include <string> 
#include <vector> 

struct foo { 
    std::vector<std::string> bar; 
}; 

const std::vector<std::string> kSample = {"test", "1", "2", "3", "4", "5"}; 

int main(int argc, char *argv[]) { 
    struct foo *allocated = new foo; 
    for (const auto& i : kSample) { 
    allocated->bar.push_back(i); 
    } 
    free(allocated); 
    return 0; 
} 

这里从运行此代码是在Valgrind的结果:

==18131== 192 bytes in 1 blocks are definitely lost in loss record 51 of 76 
==18131== at 0x6DFB: malloc (in /usr/local/Cellar/valgrind/3.9.0/lib/valgrind/vgpreload_memcheck-amd64-darwin.so) 
==18131== by 0x4728D: operator new(unsigned long) (in /usr/lib/libc++.1.dylib) 
==18131== by 0x10000283D: std::__1::__split_buffer<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >&>::__split_buffer(unsigned long, unsigned long, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >&) (in ./build/tools/test) 
==18131== by 0x100001EEC: std::__1::__split_buffer<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >&>::__split_buffer(unsigned long, unsigned long, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >&) (in ./build/tools/test) 
==18131== by 0x100001D9B: void std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > >::__push_back_slow_path<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&>(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) (in ./build/tools/test) 
==18131== by 0x100001278: main (in ./build/tools/test) 
==18131== 
==18131== LEAK SUMMARY: 
==18131== definitely lost: 192 bytes in 1 blocks 
==18131== indirectly lost: 0 bytes in 0 blocks 
==18131==  possibly lost: 0 bytes in 0 blocks 
==18131== still reachable: 0 bytes in 0 blocks 
==18131==   suppressed: 25,768 bytes in 377 blocks 
+3

'new'和'free'是两个完全不同的内存分配系统。不要合并它们。所有'new' /'new []'需要相应的'delete' /'delete []'。 – 0x499602D2 2014-08-29 03:52:53

+0

'main'可能真的只是'foo分配{{std :: begin(kSample),std :: end(kSample)}};'。 – chris 2014-08-29 03:54:11

+0

另外,你为什么用'new'创建结构? C++不是Java - 你不需要使用'new'来创建对象。另外,为什么在声明中使用'struct'关键字?你在阅读'C'书籍而不是'C++'书籍吗? 'foo分配;'这就是你需要的。 – PaulMcKenzie 2014-08-29 03:54:49

回答

5

你应该叫delete allocated;newdelete应成对使用。