Revision: 13367
Author:   [email protected]
Date:     Mon Jan 14 02:21:22 2013
Log:      Fix unhandlified value in GetOwnProperty function.

[email protected]
BUG=chromium:169723

Review URL: https://codereview.chromium.org/11879017
http://code.google.com/p/v8/source/detail?r=13367

Modified:
 /branches/bleeding_edge/src/runtime.cc

=======================================
--- /branches/bleeding_edge/src/runtime.cc      Fri Jan 11 05:13:11 2013
+++ /branches/bleeding_edge/src/runtime.cc      Mon Jan 14 02:21:22 2013
@@ -1098,14 +1098,14 @@

   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

Reply via email to