[Bug 253363] Crash when trying to change the Akregator backend settings if the module is not loaded.

[Christophe Giboudeaux]

https://bugs.kde.org/show_bug.cgi?id=253363





--- Comment #8 from Christophe Giboudeaux <cgiboudeaux gmail com>  2011-01-15 15:20:44 ---
commit b5afbb67ffaed58557e42e62c045f735ba091ad6
branch 4.4
Author: Christophe Giboudeaux <cgiboudeaux at gmail.com>
Date:   Sat Jan 15 15:06:37 2011 +0100

    Prevent Kontact from crashing when the backends list is not populated yet.
    BUG: 253363
    MERGE: 4.4

diff --git a/akregator/configuration/settings_advanced.cpp
b/akregator/configuration/settings_advanced.cpp
index 2eccd37..03d5922 100644
--- a/akregator/configuration/settings_advanced.cpp
+++ b/akregator/configuration/settings_advanced.cpp
@@ -78,9 +78,11 @@ void SettingsAdvanced::selectFactory( const QString& key )
 void SettingsAdvanced::slotConfigureStorage()
 {
     const QString key = cbBackend->itemData( cbBackend->currentIndex()
).toString();
-    Backend::StorageFactory* const factory = m_factories.value( key );
-    assert( factory );
-    factory->configure();
+    if( !key.isEmpty() ) {
+        Backend::StorageFactory* const factory = m_factories.value( key );
+        assert( factory );
+        factory->configure();
+    }
 }

 void SettingsAdvanced::slotFactorySelected( int pos )

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.