fries/quickjs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix OP_FMT_none_loc bytecode dumping (#192)

Browse source 
Commit f404980 ("Add fused get_loc0_loc1 opcode") introduced an
off-by-one (sometimes negative) array index bug because OP_get_loc1_loc1
replaced OP_get_loc0 as the first OP_FMT_none_loc opcode.
This commit is contained in:
Ben Noordhuis 2023-12-09 23:46:54 +01:00 committed by GitHub
parent f6ed206bd5
commit 4c1b9f8c7f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
quickjs.c
View file

@ -27165,7 +27165,7 @@ static void dump_byte_code(JSContext *ctx, int pass,
printf(",%u", get_u16(tab + pos + 8));
break;
case OP_FMT_none_loc:
idx = (op - OP_get_loc0) % 4;
idx = (op - OP_get_loc0_loc1) % 4;
goto has_loc;
case OP_FMT_loc8:
idx = get_u8(tab + pos);