QmlCompiler: Don't retrieve metaobjects for value and sequence types

This is not valid QML, and the generated code crashes.

Fixes: QTBUG-104092
Change-Id: If609acc2f2dc84a2e8f7c26d4d1b6c626f337cad
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Sami Shalayel <sami.shalayel@qt.io>
Reviewed-by: Andrei Golubev <andrei.golubev@qt.io>
(cherry picked from commit 2a4ec96ae00c7372b1827d3872990a47658d6ff5)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>

1 files changed, 15 insertions, 8 deletions

diff --git a/src/qmlcompiler/qqmljstyperesolver.cpp b/src/qmlcompiler/qqmljstyperesolver.cpp
index 2aebb9c5b8..5c15aba780 100644
--- a/src/qmlcompiler/qqmljstyperesolver.cpp
+++ b/src/qmlcompiler/qqmljstyperesolver.cpp
@@ -853,14 +853,21 @@ QQmlJSRegisterContent QQmlJSTypeResolver::scopedType(const QQmlJSScope::ConstPtr
             }
         }
 
-        // A plain reference to a non-singleton, non-attached type.
-        // If it's undefined, we can actually get an "instance" of it.
-        // Therefore, use a primitive value to store it.
-        // Otherwise this is a plain type reference without instance.
-        // We may still need the plain type reference for enum lookups,
-        // so store it in QJSValue for now.
-        return QQmlJSRegisterContent::create(metaObjectType(), metaObjectType(),
-                                             QQmlJSRegisterContent::MetaType, type);
+        switch (type->accessSemantics()) {
+        case QQmlJSScope::AccessSemantics::None:
+        case QQmlJSScope::AccessSemantics::Reference:
+            // A plain reference to a non-singleton, non-attached type.
+            // We may still need the plain type reference for enum lookups,
+            // Store it as QMetaObject.
+            // This only works with namespaces and object types.
+            return QQmlJSRegisterContent::create(metaObjectType(), metaObjectType(),
+                                                 QQmlJSRegisterContent::MetaType, type);
+        case QQmlJSScope::AccessSemantics::Sequence:
+        case QQmlJSScope::AccessSemantics::Value:
+            // This is not actually a type reference. You cannot get the metaobject
+            // of a value type in QML and sequences don't even have metaobjects.
+            break;
+        }
     }
 
     if (m_jsGlobalObject->hasProperty(name)) {