Reviewers: Sven Panne,
Description:
Fix unhandlified value in GetOwnProperty function.
[email protected]
BUG=chromium:169723
Please review this at https://codereview.chromium.org/11879017/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/runtime.cc
Index: src/runtime.cc
diff --git a/src/runtime.cc b/src/runtime.cc
index
b508ad3e995ed3f3a39454e69fe1633c41b88214..6a460e2412094dc2ec1fc5b7c2a02f38998b7f56
100644
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -1098,14 +1098,14 @@ static MaybeObject* GetOwnProperty(Isolate* isolate,
PropertyAttributes attrs = obj->GetLocalPropertyAttribute(*name);
if (attrs == ABSENT) return heap->undefined_value();
- AccessorPair* accessors = obj->GetLocalPropertyAccessorPair(*name);
+ Handle<AccessorPair> accessors(obj->GetLocalPropertyAccessorPair(*name));
Handle<FixedArray> elms =
isolate->factory()->NewFixedArray(DESCRIPTOR_SIZE);
elms->set(ENUMERABLE_INDEX, heap->ToBoolean((attrs & DONT_ENUM) == 0));
elms->set(CONFIGURABLE_INDEX, heap->ToBoolean((attrs & DONT_DELETE) ==
0));
- elms->set(IS_ACCESSOR_INDEX, heap->ToBoolean(accessors != NULL));
+ elms->set(IS_ACCESSOR_INDEX, heap->ToBoolean(!accessors.is_null()));
- if (accessors == NULL) {
+ if (accessors.is_null()) {
elms->set(WRITABLE_INDEX, heap->ToBoolean((attrs & READ_ONLY) == 0));
// GetProperty does access check.
Handle<Object> value = GetProperty(obj, name);
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
