From 2bde1b0193dbef84c9d95c9826aecb910d901196 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Wed, 24 Jun 2015 15:04:49 +0300 Subject: Add missing checks into popups. --- src/gui/popups/itempopup.h | 3 ++- src/gui/popups/popupmenu.cpp | 3 +++ src/gui/popups/popupmenu.h | 5 +++-- 3 files changed, 8 insertions(+), 3 deletions(-) (limited to 'src/gui/popups') diff --git a/src/gui/popups/itempopup.h b/src/gui/popups/itempopup.h index c8ff7457a..32ace3af3 100644 --- a/src/gui/popups/itempopup.h +++ b/src/gui/popups/itempopup.h @@ -75,7 +75,8 @@ class ItemPopup final : public Popup int mLastId; unsigned char mLastColor; - void setLabelColor(Label *label, const ItemType::Type type) const; + void setLabelColor(Label *label, + const ItemType::Type type) const A_NONNULL(2); }; extern ItemPopup *itemPopup; diff --git a/src/gui/popups/popupmenu.cpp b/src/gui/popups/popupmenu.cpp index f136e5bd5..10ae03365 100644 --- a/src/gui/popups/popupmenu.cpp +++ b/src/gui/popups/popupmenu.cpp @@ -968,6 +968,9 @@ void PopupMenu::showWindowPopup(Window *const window) void PopupMenu::addWindowMenu(Window *const window) { + if (!window) + return; + if (window->getCloseButton()) { // TRANSLATORS: popup menu item diff --git a/src/gui/popups/popupmenu.h b/src/gui/popups/popupmenu.h index d963591bc..ac6798ce1 100644 --- a/src/gui/popups/popupmenu.h +++ b/src/gui/popups/popupmenu.h @@ -168,7 +168,7 @@ class PopupMenu final : public Popup, public LinkHandler void addFollow(); - void addBuySell(const Being *const being); + void addBuySell(const Being *const being) A_NONNULL(2); void addParty(const std::string &nick); @@ -182,7 +182,8 @@ class PopupMenu final : public Popup, public LinkHandler void addProtection(); - void addUseDrop(const Item *const item, const bool isProtected); + void addUseDrop(const Item *const item, + const bool isProtected) A_NONNULL(2); void addGmCommands(); -- cgit v1.2.3-70-g09d2