limit toFixed() digits argument between 0 and 100 inclusive

y21 · Mar 21, 2024 · b508fa0 · b508fa0
1 parent d6168ee
commit b508fa0
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/crates/dash_vm/src/js_std/number.rs b/crates/dash_vm/src/js_std/number.rs
@@ -77,6 +77,14 @@ pub fn to_fixed(cx: CallContext) -> Result<Value, Value> {
         .map(|n| n as usize)
         .unwrap_or(0);
 
+    if decimals > 100 {
+        throw!(
+            cx.scope,
+            RangeError,
+            "toFixed() fractional digits must be between 0 and 100 inclusive"
+        )
+    };
+
     let re = format!("{num:.decimals$}");
 
     Ok(Value::String(cx.scope.intern(re.as_ref()).into()))