Merge pull request #153 from nyw0102/kalker-skillful-perlman

Fix: Reachable assertion (radix is to high)
PaddiM8 · May 21, 2024 · bcc0f94 · bcc0f94
2 parents c94c6da + 64dba08
commit bcc0f94
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/kalk/src/parser.rs b/kalk/src/parser.rs
@@ -793,6 +793,10 @@ fn string_to_num(value: &str) -> Result<KalkFloat, KalkError> {
     }
 
     let base = get_base(value)?;
+
+    if base > 36 {
+        return Err(KalkError::InvalidNumberLiteral(value.into()));
+    }
     if let Some(result) = crate::radix::parse_float_radix(&value.replace(' ', ""), base) {
         Ok(crate::float!(result))
     } else {