valgrindはallocsより多くの解放を報告します

Question

私はvalgrindの初心者ですので、何か間違っているかもしれませんが、valgrindがallocsより多くの解放を報告したらどうしたらいいですか？

はここSSCCEガット：罰金コンパイルして、あまりにも罰金を実行しますが、valgrindのは

==15925== Invalid free()/delete/delete[] 
==15925== at 0x40221EA: operator delete[](void*) (vg_replace_malloc.c:364) 
==15925== by 0x8048689: something::~something() (test.cpp:12) 
==15925== by 0x80485F5: main (test.cpp:19) 
==15925== Address 0x42b7028 is 0 bytes inside a block of size 4 free'd 
==15925== at 0x40221EA: operator delete[](void*) (vg_replace_malloc.c:364) 
==15925== by 0x8048689: something::~something() (test.cpp:12) 
==15925== by 0x80485E5: main (test.cpp:18) 
==15925== 
==15925== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 18 from 1) 
==15925== malloc/free: in use at exit: 0 bytes in 0 blocks. 
==15925== malloc/free: 1 allocs, 2 frees, 4 bytes allocated. 
==15925== For counts of detected errors, rerun with: -v 
==15925== All heap blocks were freed -- no leaks are possible. 

言うと、私はなぜわからないんだ

#include <cstring> 

class something 
{ 
protected: 
    char* ptr; 
public: 
    something() {ptr = NULL;} 
    something(const char* value) { 
     ptr = new char[strlen(value)+1]; strcpy(ptr, value); 
    } 
    ~something() {delete[] ptr; ptr = NULL;} 
}; 

int main() 
{ 
    something x; 
    x = "123"; 
    return 0; 
} 

。
もちろん、私は教育的な推測をすることができます - 明らかに違反行は、それがx = "123";と言う場所であり、あなたがそれをコメントすれば、すべてが良いです。しかし、なぜコンパイラは-Wall -Wextra -pedanticを使っても、これが問題ないと思いますか？私はこのプログラムに問題があることを伝えるコンパイラスイッチを忘れてしまったのでしょうか？

Answer 1

Rule of threeを実装するのを忘れました。
コピーコンストラクタとコピー代入演算子を指定する必要があります。

x = "123"; 

オブジェクトの簡易コピーを作成し、暗黙的に生成されたコピー代入演算子を呼び出して、一時的に破壊されます一度デストラクタはダングリングポインタとしてポインタのメンバーを残して割り当てられたメモリの割り当てを解除します。

Answer 2

x = "123"は、暗黙のコピー代入演算子を呼び出すx = something("123")に相当します。一時的なsomethingは破壊されますが、xであり、どちらもdelete []の同じ生ポインタです。

解決策は、Rule of Threeに従うか、スマートポインタ/コンテナを使用してメモリ管理を行うことです。

Answer 3

somethingクラスには、（コンパイラ提供の）コピーコンストラクタと代入演算子で適切に管理されていないローポインタが含まれています。生ポインタを使用しないでください。必要がある場合は、クラスメソッドをより慎重に定義してください。しかし実際には、生ポインタを使用しないでください。