admin/mruby-mruby
1
0
Fork 0
mirror of https://github.com/mruby/mruby synced 2026-06-08 16:11:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

mruby-io/file.c: atime, ctime, mtime check mrb_int overflow

Browse Source 
When `sizeof(time_t)` is different from `mrb_int`, overflow may happen.
This commit is contained in:
Yukihiro "Matz" Matsumoto
2023-03-03 14:21:05 +09:00
parent 0e51da81bb
commit 5237b2cccc
1 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-io/src/file.c
+11
View File
@@ -391,6 +391,8 @@ mrb_file__gethome(mrb_state *mrb, mrb_value klass)
#endif
}
#define TIME_OVERFLOW_P(a) (sizeof(time_t) <= sizeof(mrb_int) && (a > MRB_INT_MAX || a < MRB_INT_MIN))
static mrb_value
mrb_file_atime(mrb_state *mrb, mrb_value self)
{
@@ -399,6 +401,9 @@ mrb_file_atime(mrb_state *mrb, mrb_value self)
if (mrb_fstat(fd, &st) == -1)
mrb_sys_fail(mrb, "atime");
if (TIME_OVERFLOW_P(st.st_atime)) {
mrb_raise(mrb, E_IO_ERROR, "atime overflow");
}
return mrb_int_value(mrb, (mrb_int)st.st_atime);
}
@@ -410,6 +415,9 @@ mrb_file_ctime(mrb_state *mrb, mrb_value self)
if (mrb_fstat(fd, &st) == -1)
mrb_sys_fail(mrb, "ctime");
if (TIME_OVERFLOW_P(st.st_ctime)) {
mrb_raise(mrb, E_IO_ERROR, "ctime overflow");
}
return mrb_int_value(mrb, (mrb_int)st.st_ctime);
}
@@ -421,6 +429,9 @@ mrb_file_mtime(mrb_state *mrb, mrb_value self)
if (mrb_fstat(fd, &st) == -1)
mrb_sys_fail(mrb, "mtime");
if (TIME_OVERFLOW_P(st.st_mtime)) {
mrb_raise(mrb, E_IO_ERROR, "mtime overflow");
}
return mrb_int_value(mrb, (mrb_int)st.st_mtime);
}