Comment 6 for bug 737713

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Got a patch done; since it's not an issue that will gravely affect the release I think it can wait until checked out properly by dcbw first, before upload.

Essentially, clean up the VPN connections and disconnect signals from the activation requests before unref'ing vpn_manager; otherwise the signals get received and the callback for them tries to ask the NMManager object for stuff, but the object has already been cleared.

Why I'm saying this won't gravely affect the release is because it's a crash that happens when NM is stopped (which means it ends up stopped, ie not running anymore, anyway), it still starts up fine after, and we won't be showing apport dialogs past FinalFreeze. Regardless, we'll still be fixing this.