From aa0d22a129fdcc07818e311e031a167b6cce8b65 Mon Sep 17 00:00:00 2001 From: theadib Date: Wed, 20 Feb 2008 22:14:48 +0000 Subject: [PATCH 1/1] pointer check, to prevent rapid crash on pdf/ps-export --- src/extension/prefdialog.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/extension/prefdialog.cpp b/src/extension/prefdialog.cpp index 5a3204967..280269ec2 100644 --- a/src/extension/prefdialog.cpp +++ b/src/extension/prefdialog.cpp @@ -33,7 +33,7 @@ namespace Extension { \param name Name of the Extension who's dialog this is \param help The help string for the extension (NULL if none) \param controls The extension specific widgets in the dialog - + This function initializes the dialog with the name of the extension in the title. It adds a few buttons and sets up handlers for them. It also places the passed in widgets into the dialog. @@ -68,8 +68,8 @@ PrefDialog::PrefDialog (Glib::ustring name, gchar const * help, Gtk::Widget * co _button_ok->set_use_stock(true); set_default_response(Gtk::RESPONSE_OK); _button_ok->grab_focus(); - - // If we're working with an effect that can be live and + + // If we're working with an effect that can be live and // the dialog can be pinned, put those options in too if (_exEnv != NULL) { if (_param_preview == NULL) { @@ -173,7 +173,10 @@ PrefDialog::on_response (int signal) { } if (signal == Gtk::RESPONSE_OK) { - _effect->effect(SP_ACTIVE_DESKTOP); + if(_effect != NULL) + { + _effect->effect(SP_ACTIVE_DESKTOP); + } } if (signal == Gtk::RESPONSE_CANCEL) { // close the dialog -- 2.30.2