Do not take a reference of nullptr

References are not allowed to be null, but we pass a nullptr as
receiver in QQmlContextWrapper::resolveQmlContextPropertyLookupGetter.

Detected with UBSAN.

Change-Id: Iaa7945fb17e4b0e549e541e47589b2f47d32ea4e
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

1 files changed, 4 insertions, 4 deletions

diff --git a/src/qml/jsruntime/qv4object_p.h b/src/qml/jsruntime/qv4object_p.h
index c3f1cb2c35..bee4aadafe 100644
--- a/src/qml/jsruntime/qv4object_p.h
+++ b/src/qml/jsruntime/qv4object_p.h
@@ -185,22 +185,22 @@ struct Q_QML_EXPORT Object: Managed {
     //
     // helpers
     //
-    static ReturnedValue getValue(const Value &thisObject, const Value &v, PropertyAttributes attrs) {
+    static ReturnedValue getValue(const Value *thisObject, const Value &v, PropertyAttributes attrs) {
         if (attrs.isData())
             return v.asReturnedValue();
         return getValueAccessor(thisObject, v, attrs);
     }
     ReturnedValue getValue(const Value &v, PropertyAttributes attrs) const {
-        return getValue(*this, v, attrs);
+        return getValue(this, v, attrs);
     }
     ReturnedValue getValueByIndex(uint propertyIndex) const {
         PropertyAttributes attrs = internalClass()->propertyData.at(propertyIndex);
         const Value *v = propertyData(propertyIndex);
         if (!attrs.isAccessor())
             return v->asReturnedValue();
-        return getValueAccessor(*this, *v, attrs);
+        return getValueAccessor(this, *v, attrs);
     }
-    static ReturnedValue getValueAccessor(const Value &thisObject, const Value &v, PropertyAttributes attrs);
+    static ReturnedValue getValueAccessor(const Value *thisObject, const Value &v, PropertyAttributes attrs);
 
     bool putValue(uint memberIndex, PropertyAttributes attrs, const Value &value);