Seltsame doppelte Destruktor Aufruf bei der Verwendung von shared_ptr

Question

Endlich habe ich sehr seltsamen Bug, der durch doppelte Aufruf Destruktor verursacht wird, aufgespürt. Hier ist der minimale Code, der den Fehler reproduziert:

#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); 
} 

Die Ausgabe lautet:

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

Wie Sie sehen, der erste destructor ist unerwünscht und wird auf verschiedenen Objekt aufgerufen, die bei nicht gebaut wurde alle (die Adresse ist anders), aber in meinem echten Code ist die Adresse nah genug und es verdirbt die Container, die ich im Event-System habe.

Debugging zeigt, dass make_shared den Destruktoraufruf verursacht.

Was verursacht diesen unerwünschten Destruktoranruf und wie kann ich ihn loswerden? Ich benutze g ++ 4.7 mit C++ 11 Flagge.

Das Problem ist, dass die unerwünschte Destruktoraufrufs in der Regel (90% der Zeit) verdirbt mein Ereignissystem Container in meinem realen Code, die segfaults verursacht, aber es funktioniert nicht selten verderben und alles funktioniert.

Answer 1

Der CTileBrowser-Konstruktor nimmt sein Argument nach Wert. Sie sehen wahrscheinlich die Zerstörung einer temporären Kopie, die für diesen Konstruktor erstellt wurde. Ändern Sie es zu einem Referenzparameter und ich wette, das Problem wird verschwinden.