Merge pull request #13668 from acolombier/fix/qml-quickfx-model

fix: prevent quickFX model out of bound
mixxxdj · Sep 19, 2024 · f494727 · f494727
2 parents 6b4a6f9 + 51b5d02
commit f494727
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/qml/qmlchainpresetmodel.cpp b/src/qml/qmlchainpresetmodel.cpp
@@ -62,8 +62,13 @@ QHash<int, QByteArray> QmlChainPresetModel::roleNames() const {
 }
 
 QVariant QmlChainPresetModel::get(int row) const {
-    QModelIndex idx = index(row, 0);
     QVariantMap dataMap;
+    QModelIndex idx = index(row, 0);
+
+    if (!idx.isValid()) {
+        return dataMap;
+    }
+
     for (auto it = kRoleNames.constBegin(); it != kRoleNames.constEnd(); it++) {
         dataMap.insert(it.value(), data(idx, it.key()));
     }