Merge pull request #1739 from o1-labs/fix-prefix

Limit prefixToField to only accept ascii characters
o1-labs · Jul 16, 2024 · 6be263c · 6be263c
2 parents 745e863 + e6ac07a
commit 6be263c
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/lib/provable/crypto/poseidon.ts b/src/lib/provable/crypto/poseidon.ts
@@ -160,6 +160,11 @@ function prefixToField(prefix: string) {
       // convert char to 8 bits
       let bits = [];
       for (let j = 0, c = char.charCodeAt(0); j < 8; j++, c >>= 1) {
+        if (j === 7)
+          assert(
+            c === 0,
+            `Invalid character ${char}, only ASCII characters are supported.`
+          );
         bits.push(!!(c & 1));
       }
       return bits;