summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorYukihiro "Matz" Matsumoto <[email protected]>2021-09-07 07:35:35 +0900
committerYukihiro "Matz" Matsumoto <[email protected]>2021-09-07 07:35:35 +0900
commit711f06a562117c63b5b7b04f79d3c4a6b076b098 (patch)
tree3f1e91c3d68ad2770ee3a18e1f3e7c5832f34b8a
parent6fc6880f1fa84991e6b4d7e9b961d34568835dd5 (diff)
downloadmruby-711f06a562117c63b5b7b04f79d3c4a6b076b098.tar.gz
mruby-711f06a562117c63b5b7b04f79d3c4a6b076b098.zip
eval.c: check length of the file name.
It should be lexx than `UINT16_MAX`. If you don't check here, the parser would raise an exception.
-rw-r--r--mrbgems/mruby-eval/src/eval.c11
1 files changed, 10 insertions, 1 deletions
diff --git a/mrbgems/mruby-eval/src/eval.c b/mrbgems/mruby-eval/src/eval.c
index b10e0c7ce..a3c421f45 100644
--- a/mrbgems/mruby-eval/src/eval.c
+++ b/mrbgems/mruby-eval/src/eval.c
@@ -52,10 +52,19 @@ create_proc_from_string(mrb_state *mrb, const char *s, mrb_int len, mrb_value bi
e = NULL;
}
+ if (file) {
+ if (strlen(file) >= UINT16_MAX) {
+ mrb_raise(mrb, E_ARGUMENT_ERROR, "filename too long");
+ }
+ }
+ else {
+ file = "(eval)";
+ }
+
cxt = mrbc_context_new(mrb);
cxt->lineno = (uint16_t)line;
- mrbc_filename(mrb, cxt, file ? file : "(eval)");
+ mrbc_filename(mrb, cxt, file);
cxt->capture_errors = TRUE;
cxt->no_optimize = TRUE;
cxt->upper = scope && MRB_PROC_CFUNC_P(scope) ? NULL : scope;