Merge pull request #4996 from dearblue/loadi16

Fix boundary check for `OP_LOADI16`; ref fa8668c7

Merge pull request #4996 from dearblue/loadi16
Fix boundary check for `OP_LOADI16`; ref fa8668c7
84d842ea · Yukihiro "Matz" Matsumoto · GitHub · 8cdf6a87 · e2aecaca · 84d842ea
Unverified Commit 84d842ea authored May 09, 2020 by Yukihiro "Matz" Matsumoto Committed by GitHub May 09, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

mrbgems/mruby-compiler/core/codegen.c mrbgems/mruby-compiler/core/codegen.c +1 -1

No files found.
--- a/mrbgems/mruby-compiler/core/codegen.c
+++ b/mrbgems/mruby-compiler/core/codegen.c
@@ -2448,7 +2448,7 @@ codegen(codegen_scope *s, node *tree, int val)
        }
        else if (i < 8) genop_1(s, OP_LOADI_0 + (uint8_t)i, cursp());
        else if (i <= 0xff) genop_2(s, OP_LOADI, cursp(), (uint16_t)i);
-        else if (i <= 0xffff) genop_2S(s, OP_LOADI16, cursp(), (uint16_t)i);
+        else if (i <= 0x7fff) genop_2S(s, OP_LOADI16, cursp(), (uint16_t)i);
        else {
          int off;