From c9d57b2249aadc6440bbac97f32ad83024b91f3c Mon Sep 17 00:00:00 2001 From: Jedrzej Nowacki Date: Thu, 4 Aug 2011 13:20:12 +0200 Subject: [PATCH] Add missing check in QJSValue::property Only an object can have a property. Change-Id: I032cf3a08f777b6b47cb5a417fa796059bfc9ef5 Reviewed-on: http://codereview.qt.nokia.com/2702 Reviewed-by: Qt Sanity Bot Reviewed-by: Simon Hausmann --- src/declarative/qml/v8/qjsvalue_impl_p.h | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/declarative/qml/v8/qjsvalue_impl_p.h b/src/declarative/qml/v8/qjsvalue_impl_p.h index 414109c..39bfd89 100644 --- a/src/declarative/qml/v8/qjsvalue_impl_p.h +++ b/src/declarative/qml/v8/qjsvalue_impl_p.h @@ -826,6 +826,8 @@ inline QScriptPassPointer QJSValuePrivate::property(const QStri { if (!name.length()) return InvalidValue(); + if (!isObject()) + return InvalidValue(); v8::HandleScope handleScope; return property(QJSConverter::toString(name)); -- 1.7.2.5