From 756249c3ad0e9514e5fea731bd7085784eecad33 Mon Sep 17 00:00:00 2001
From: Dan Paulat <dpaulat@gmail.com>
Date: Tue, 26 Nov 2024 05:53:14 -0600
Subject: [PATCH] Validate parent row before dropping mime data in LayerModel

---
 scwx-qt/source/scwx/qt/model/layer_model.cpp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/scwx-qt/source/scwx/qt/model/layer_model.cpp b/scwx-qt/source/scwx/qt/model/layer_model.cpp
index 23d05cd6..014acb42 100644
--- a/scwx-qt/source/scwx/qt/model/layer_model.cpp
+++ b/scwx-qt/source/scwx/qt/model/layer_model.cpp
@@ -774,6 +774,13 @@ bool LayerModel::dropMimeData(const QMimeData* data,
    QDataStream      stream(&mimeData, QIODevice::ReadOnly);
    std::vector<int> sourceRows {};
 
+   // Validate parent row
+   if (parent.row() < 0 || parent.row() >= static_cast<int>(p->layers_.size()))
+   {
+      logger_->warn("Cannot perform drop action, invalid parent row");
+      return false;
+   }
+
    // Read source rows from QMimeData
    while (!stream.atEnd())
    {