fix: DefineOwnProperty 臂写参数寄存器补顶帧守卫

This commit is contained in:
2026-10-10 00:12:44 +00:00
parent c7f7588b81
commit b6219188e7
7 changed files with 71 additions and 24 deletions
+10 -4
View File
@@ -984,8 +984,9 @@ pub(crate) fn define_from_descriptor<H: VmHost>(
}
// mapped arguments exotic [[DefineOwnProperty]]:整数索引且映射存活时,旧描述符
// 可写且新描述符不可写则移除映射,新描述符含 value 则写参数寄存器;随后落穿
// 普通 define 路径(描述符收窄校验由 define_data_property 承担)。
// 可写且新描述符不可写则移除映射,新描述符含 value 且创建帧为顶帧时写参数
// 寄存器;随后落穿普通 define 路径(描述符收窄校验由 define_data_property
// 承担)。
if unsafe { &*obj_ptr }.is_arguments_obj() {
let state_ptr = unsafe { &*obj_ptr }.native_data() as *mut ArgumentsMapState;
if !state_ptr.is_null() {
@@ -1006,8 +1007,13 @@ pub(crate) fn define_from_descriptor<H: VmHost>(
state.unmap(index as u16);
}
if let Some(v) = value_field {
let reg = (state.param_base as u16 + index as u16) as u8;
vm.set_reg(reg, v);
// 参数寄存器仅创建帧为顶帧时写——嵌套调用期寄存器被被调方
// 占用,写之串值,参数值在 save_stack,存储值仍为权威
// 读源(与 [[Set]] 臂同口径)。
if vm.is_top_frame(state.frame_depth) {
let reg = (state.param_base as u16 + index as u16) as u8;
vm.set_reg(reg, v);
}
}
}
}
+5
View File
@@ -259,6 +259,11 @@ pub trait VmHost {
/// - `state` 为 arguments 对象 `native_data` 中的同步状态盒;`frame_depth`
/// 为创建帧下标,帧弹出后映射失效。
fn arguments_mapping_alive(&self, state: &oxide_types::arguments_map::ArgumentsMapState) -> bool;
/// 判定帧栈下标 `frame_depth` 处的帧是否为顶帧(当前帧)。
///
/// # 边界与前提
/// - `frame_depth` 为创建帧在帧栈中的下标;帧弹出后下标越界返回 `false`。
fn is_top_frame(&self, frame_depth: u32) -> bool;
// 属性访问
fn ordinary_get(&mut self, obj: &JsObject, prop_name_si: u32, receiver: JsValue) -> Result<JsValue, String>;
+3
View File
@@ -218,6 +218,9 @@ impl oxide_runtime_api::VmHost for Vm {
fn arguments_mapping_alive(&self, state: &oxide_types::arguments_map::ArgumentsMapState) -> bool {
self.arguments_mapping_alive(state)
}
fn is_top_frame(&self, frame_depth: u32) -> bool {
self.is_top_frame(frame_depth)
}
fn ordinary_get(&mut self, obj: &JsObject, prop_name_si: u32, receiver: JsValue) -> Result<JsValue, String> {
self.ordinary_get(obj, prop_name_si, receiver)
}
+8
View File
@@ -20,6 +20,14 @@ impl Vm {
self.frames.len() > depth && self.frames[depth].frame_id == state.frame_id
}
/// 判定帧栈下标 `frame_depth` 处的帧是否为顶帧(当前帧)。
///
/// # 边界与前提
/// - `frame_depth` 为创建帧在帧栈中的下标;帧弹出后下标越界返回 `false`。
pub(crate) fn is_top_frame(&self, frame_depth: u32) -> bool {
self.frames.len() == frame_depth as usize + 1
}
/// 属性读入口:解析 `obj[prop_name_si]`,依次尝试数组 length 虚拟属性、
/// 数组元素区、TypedArray 整数索引、命名属性槽,最后沿原型链查找。
///
+25
View File
@@ -204,3 +204,28 @@ fn global_self_decrement_preserves_arguments() {
let result = eval(&mut vm, "var c=1; function f(a){c--; return arguments.length} f(1)").unwrap();
assert_eq!(result.as_int(), 1);
}
#[test]
fn define_property_nested_call_does_not_clobber_callee_param() {
// 嵌套调用期对外层函数 arguments 索引属性 defineProperty 不写被调方帧的
// 参数寄存器(顶帧守卫):被调方形参值不被串改。
let mut vm = Vm::new();
let result = eval(
&mut vm,
"function a(x){function b(y){Object.defineProperty(a.arguments,0,{value:5}); return y} return b(7)} a(1)",
)
.unwrap();
assert_eq!(result.as_int(), 7);
}
#[test]
fn define_property_nested_call_updates_stored_value() {
// 嵌套调用期 defineProperty 更新存储值(读路径权威源):回到顶帧后读为新值。
let mut vm = Vm::new();
let result = eval(
&mut vm,
"function a(x){function b(){Object.defineProperty(a.arguments,0,{value:5})} b(); return a.arguments[0]} a(1)",
)
.unwrap();
assert_eq!(result.as_int(), 5);
}