config dialog: Protect against null pages add.
authorLuís Pereira <luis.artur.pereira@gmail.com>
Mon, 19 Oct 2015 09:14:44 +0000 (10:14 +0100)
committerLuís Pereira <luis.artur.pereira@gmail.com>
Mon, 19 Oct 2015 09:14:44 +0000 (10:14 +0100)
Make it resilient to null pointers pages.

configdialog/lxqtconfigdialog.cpp

index e469d94..6217761 100644 (file)
@@ -59,6 +59,12 @@ void ConfigDialog::addPage(QWidget* page, const QString& name, const QString& ic
 
 void ConfigDialog::addPage(QWidget* page, const QString& name, const QStringList& iconNames)
 {
+    Q_ASSERT(page);
+    if (!page)
+    {
+        return;
+    }
+
     /* We set the layout margin to 0. In the default configuration, one page
      *  only, it aligns buttons with the page. In multi-page it saves a little
      *  bit of space, without clutter.