Prevent null pointer deference in methods for selection.

This resolves the crash when exporting the Basket archive.

Signed-off-by: Slávek Banko <slavek.banko@axis.cz>

1 files changed, 12 insertions, 0 deletions

diff --git a/src/basket.cpp b/src/basket.cpp
index 6e14c65..128c1cb 100644
--- a/src/basket.cpp
+++ b/src/basket.cpp
@@ -4469,12 +4469,20 @@ void Basket::noteUngroup()
 
 void Basket::unplugSelection(NoteSelection *selection)
 {
+	if (!selection)
+	{
+		return;
+	}
 	for (NoteSelection *toUnplug = selection->firstStacked(); toUnplug; toUnplug = toUnplug->nextStacked())
 		unplugNote(toUnplug->note);
 }
 
 void Basket::insertSelection(NoteSelection *selection, Note *after)
 {
+	if (!selection)
+	{
+		return;
+	}
 	for (NoteSelection *toUnplug = selection->firstStacked(); toUnplug; toUnplug = toUnplug->nextStacked()) {
 		if (toUnplug->note->isGroup()) {
 			Note *group = new Note(this);
@@ -4496,6 +4504,10 @@ void Basket::insertSelection(NoteSelection *selection, Note *after)
 
 void Basket::selectSelection(NoteSelection *selection)
 {
+	if (!selection)
+	{
+		return;
+	}
 	for (NoteSelection *toUnplug = selection->firstStacked(); toUnplug; toUnplug = toUnplug->nextStacked()) {
 		if (toUnplug->note->isGroup())
 			selectSelection(toUnplug);