wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 957688 - Remove side-effect-y call to js::CheckAccess, and remove js::CheckAccess itself. r=mrbkap

Browse Source 
Thankfully, this case was only taking the JSACC_PROTO, which is significantly
simpler than the alternative.
This commit is contained in:
Bobby Holley 2014-01-24 16:08:25 -08:00
parent 3bf47f1f26
commit abb3c5c1c7
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
js/src/vm/GlobalObject.cpp
View File

@ -82,14 +82,11 @@ ProtoGetterImpl(JSContext *cx, CallArgs args)
if (thisv.isPrimitive() && !BoxNonStrictThis(cx, args))
return false;
unsigned dummy;
RootedObject obj(cx, &args.thisv().toObject());
RootedId nid(cx, NameToId(cx->names().proto));
RootedValue v(cx);
if (!CheckAccess(cx, obj, nid, JSACC_PROTO, &v, &dummy))
RootedObject proto(cx);
if (!JSObject::getProto(cx, obj, &proto))
return false;
args.rval().set(v);
args.rval().setObjectOrNull(proto);
return true;
}