Fix parse_print_final_cbc dump when arguments is used

zherczeg · zherczeg · commit 1bb30aa96119 · 2018-06-18T03:05:42.000-07:00
Currently the byte code start is incorrectly set when
a non-strict arguments object is present, and a random
memory area is dumped as byte code.

JerryScript-DCO-1.0-Signed-off-by: Zoltan Herczeg zherczeg.u-szeged@partner.samsung.com
diff --git a/jerry-core/parser/js/js-parser.c b/jerry-core/parser/js/js-parser.c
@@ -1228,10 +1228,6 @@ parse_print_final_cbc (ecma_compiled_code_t *compiled_code_p, /**< compiled code
   }
 
   byte_code_start_p += (unsigned int) (literal_end - register_end) * sizeof (ecma_value_t);
-  if (JERRY_UNLIKELY (compiled_code_p->status_flags & CBC_CODE_FLAGS_NON_STRICT_ARGUMENTS_NEEDED))
-  {
-    byte_code_start_p += argument_end * sizeof (ecma_value_t);
-  }
 
   byte_code_end_p = byte_code_start_p + length;
   byte_code_p = byte_code_start_p;

Original file line number	Diff line number	Diff line change
`@@ -1228,10 +1228,6 @@ parse_print_final_cbc (ecma_compiled_code_t compiled_code_p, /*< compiled code`
`1228`	`1228`	`}`
`1229`	`1229`
`1230`	`1230`	`byte_code_start_p += (unsigned int) (literal_end - register_end) * sizeof (ecma_value_t);`
`1231`		`- if (JERRY_UNLIKELY (compiled_code_p->status_flags & CBC_CODE_FLAGS_NON_STRICT_ARGUMENTS_NEEDED))`
`1232`		`- {`
`1233`		`- byte_code_start_p += argument_end * sizeof (ecma_value_t);`
`1234`		`- }`
`1235`	`1231`
`1236`	`1232`	`byte_code_end_p = byte_code_start_p + length;`
`1237`	`1233`	`byte_code_p = byte_code_start_p;`