Revision: 6618 Author: [email protected] Date: Thu Feb 3 05:30:22 2011 Log: Fix array boundary error.
It made value of allowed_access_type[v8::ACCESS_KEYS] be a wild value. On most of platforms it was 0 and tests passed. But on ARM (and on ia32 if you alter test a bit)
it could become true and hence allow enumeration of properties. Review URL: http://codereview.chromium.org/6334089 http://code.google.com/p/v8/source/detail?r=6618 Modified: /branches/bleeding_edge/test/cctest/test-api.cc ======================================= --- /branches/bleeding_edge/test/cctest/test-api.cc Thu Feb 3 04:50:50 2011 +++ /branches/bleeding_edge/test/cctest/test-api.cc Thu Feb 3 05:30:22 2011 @@ -5309,7 +5309,7 @@ } -static bool allowed_access_type[v8::ACCESS_KEYS] = { false }; +static bool allowed_access_type[v8::ACCESS_KEYS + 1] = { false }; static bool NamedAccessBlocker(Local<v8::Object> global, Local<Value> name, v8::AccessType type, -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
