C++ strcpy 后缓冲区的错误

Question

这是我的代码片段

    std::string s;  
    getline(cin, s);
    std::cin.clear();
    s.erase(std::remove_if(s.begin(), s.end(), my_predicate), s.end());
    char *str1  = (char*)malloc(sizeof(char)*(s.size()+1));
    const int length = s.length();
    for(int i=0; i < length; ++i)
    {
        s[i] = std::tolower(s[i]);
    }
    strncpy(str1,s.c_str(), s.length()+1);
    printf("n");

问题是如果我不打印，代码就会崩溃。我也尝试过memcpy和strdup，但无济于事。是否有任何错误或任何方法可以摆脱此????

Answer 1

如果您绝对需要std::string s的可写副本，请检查相关问题的答案;

而不是

char *str1  = (char*)malloc(sizeof(char)*(s.size()+1));
strncpy(str1,s.c_str(), s.length()+1);

基本上，这样做：

boost::scoped_array<char> writable(new char[s.size() + 1]);
std::copy(s.begin(), s.end(), writable.get());
writable[s.size()] = ''; 
// access char * via writable.get()!

这带来了额外的好处，即不必担心之后是否释放任何内存！

Answer 2

虽然我认为你的代码并不完美，但我不明白为什么它会崩溃。既然你说，printf（""） 防止崩溃，我想堆已经损坏了。也许显示 myPredicate 中的代码或独立测试给定的代码片段以找到原因。