DoneJS/quickjs-done
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix non-ASCII strings in JS_NewAtomLen

Browse source
This commit is contained in:
Emil Lenngren 2024-06-09 16:02:07 +02:00
parent 36911f0d3a
commit b049cf0704
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
quickjs.c
View file

@ -2964,11 +2964,16 @@ JSAtom JS_NewAtomLen(JSContext *ctx, const char *str, size_t len)
JSValue val; JSValue val;
if (len == 0 || !is_digit(*str)) { if (len == 0 || !is_digit(*str)) {
// XXX: this will not work if UTF-8 encoded str contains non ASCII bytes size_t i;
for(i = 0; i < len; i++) {
if ((uint8_t)str[i] >= 0x80)
goto slow_path;
}
JSAtom atom = __JS_FindAtom(ctx->rt, str, len, JS_ATOM_TYPE_STRING); JSAtom atom = __JS_FindAtom(ctx->rt, str, len, JS_ATOM_TYPE_STRING);
if (atom) if (atom)
return atom; return atom;
} }
slow_path:
val = JS_NewStringLen(ctx, str, len); val = JS_NewStringLen(ctx, str, len);
if (JS_IsException(val)) if (JS_IsException(val))
return JS_ATOM_NULL; return JS_ATOM_NULL;