shared_ptrを使用すると奇妙なダブルデストラクタコール

Question

最後に、デストラクタを二重に呼び出すことによって引き起こされた非常に奇妙なバグを追跡しました。ここでバグを再現する最小限のコードは次のとおりです。

#include <iostream> 
#include <memory> 
#include <set> 

class cEventSystem { 
    public: 
     cEventSystem() { 
      std::cout << "constructor: " << this << std::endl; 
     } 
     ~cEventSystem() { 
      std::cout << "destructor: " << this << std::endl; 
     } 
}; 

class cSubscriber { 
    public: 
     cSubscriber(cEventSystem& eventSystem) : eventSystem(eventSystem) {} 
     virtual ~cSubscriber() {} 
     virtual void onEvent() = 0; 
    protected: 
     cEventSystem& eventSystem; 
}; 

class cTileBrowser: public cSubscriber { 
    public: 
     cTileBrowser(cEventSystem eventSystem) : cSubscriber(eventSystem) {} 
     void onEvent() {} 
}; 

class cGui: public cSubscriber { 
    public: 
     cGui(cEventSystem& eventSystem) : cSubscriber(eventSystem) { 
      tileBrowser = std::make_shared<cTileBrowser>(eventSystem); 
     } 
     void onEvent() {} 
     std::shared_ptr<cTileBrowser> tileBrowser; 
}; 

int main() { 
    cEventSystem eventSystem; 
    cGui gui(eventSystem); 
} 

出力は次のとおりです。

constructor: 0x7fffffffe67f 
destructor: 0x7fffffffe2df 
destructor: 0x7fffffffe67f 

あなたが最初のデストラクタが不要であり、それがで構築されていなかった別のオブジェクトで呼び出される見ることができるようにすべて（住所は違う）、私の実際のコードでは住所は十分に近く、私がイベントシステムに持っているコンテナを壊す。

デバッグは、そのデストラクタコールを引き起こすmake_sharedであることを示しています。

この不要なデストラクタコールはどのような原因で削除されますか？ g ++ 4.7とC++ 11フラグを使用します。

通常は（90％の時間）は、segfaultsの原因となる実際のコードで自分のイベントシステムコンテナを破損しますが、まれに破損せずすべてが機能します。

Answer 1

CTileBrowserコンストラクタは値で引数を受け取ります。あなたは、そのコンストラクタのために作成された一時的なコピーが破損している可能性が高いです。それを参照パラメータに変更して、問題がなくなると思います。