Fix logic error in JSCodeGen::fallbackNameLookup()

Don't return a name reference if we fail to lookup something in
the scope object, but rather continue trying in the context
object.

Task-number: QTBUG-70315
Change-Id: I73f8aa7b648320434ef0ef37a4c12dca1eb7b209
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

1 files changed, 22 insertions, 26 deletions

diff --git a/src/qml/compiler/qqmlirbuilder.cpp b/src/qml/compiler/qqmlirbuilder.cpp
index 88accc2f49..883b21ab07 100644
--- a/src/qml/compiler/qqmlirbuilder.cpp
+++ b/src/qml/compiler/qqmlirbuilder.cpp
@@ -2279,42 +2279,38 @@ QV4::Compiler::Codegen::Reference JSCodeGen::fallbackNameLookup(const QString &n
 
     if (_scopeObject) {
         QQmlPropertyData *data = lookupQmlCompliantProperty(_scopeObject, name);
-        if (!data)
-            return Reference::fromName(this, name);
-
-        Reference base = Reference::fromStackSlot(this, _qmlContextSlot);
-        Reference::PropertyCapturePolicy capturePolicy;
-        if (!data->isConstant() && !data->isQmlBinding())
-            capturePolicy = Reference::CaptureAtRuntime;
-        else
-            capturePolicy = data->isConstant() ? Reference::DontCapture : Reference::CaptureAheadOfTime;
-        return Reference::fromQmlScopeObject(base, data->coreIndex(), data->notifyIndex(), capturePolicy);
+        if (data) {
+            Reference base = Reference::fromStackSlot(this, _qmlContextSlot);
+            Reference::PropertyCapturePolicy capturePolicy;
+            if (!data->isConstant() && !data->isQmlBinding())
+                capturePolicy = Reference::CaptureAtRuntime;
+            else
+                capturePolicy = data->isConstant() ? Reference::DontCapture : Reference::CaptureAheadOfTime;
+            return Reference::fromQmlScopeObject(base, data->coreIndex(), data->notifyIndex(), capturePolicy);
+        }
     }
 
     if (_contextObject) {
         QQmlPropertyData *data = lookupQmlCompliantProperty(_contextObject, name);
-        if (!data)
-            return Reference::fromName(this, name);
-
-        Reference base = Reference::fromStackSlot(this, _qmlContextSlot);
-        Reference::PropertyCapturePolicy capturePolicy;
-        if (!data->isConstant() && !data->isQmlBinding())
-            capturePolicy = Reference::CaptureAtRuntime;
-        else
-            capturePolicy = data->isConstant() ? Reference::DontCapture : Reference::CaptureAheadOfTime;
-        return Reference::fromQmlContextObject(base, data->coreIndex(), data->notifyIndex(), capturePolicy);
+        if (data) {
+            Reference base = Reference::fromStackSlot(this, _qmlContextSlot);
+            Reference::PropertyCapturePolicy capturePolicy;
+            if (!data->isConstant() && !data->isQmlBinding())
+                capturePolicy = Reference::CaptureAtRuntime;
+            else
+                capturePolicy = data->isConstant() ? Reference::DontCapture : Reference::CaptureAheadOfTime;
+            return Reference::fromQmlContextObject(base, data->coreIndex(), data->notifyIndex(), capturePolicy);
+        }
     }
 
-    if (m_globalNames.contains(name)) {
-        Reference r = Reference::fromName(this, name);
+    Reference r = Reference::fromName(this, name);
+    if (m_globalNames.contains(name))
         r.global = true;
-        return r;
-    }
+    return r;
 #else
     Q_UNUSED(name)
-#endif // V4_BOOTSTRAP
-    // fall back to name lookup at run-time.
     return Reference();
+#endif // V4_BOOTSTRAP
 }
 
 #ifndef V4_BOOTSTRAP