diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-05-24 14:01:34 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-05-24 14:01:34 +0000 |
commit | a4a922e9bce0ba129d458f340b603a7a8c4b0bc0 (patch) | |
tree | a6b859087713f153776e0631c73ef61478b3a3df /vm.c | |
parent | 04a03b34c3b5320c23d9e78f4acc884370e01bb1 (diff) |
vm_eval.c: fix types
* vm_eval.c (rb_backtrace_struct, backtreace_collect): use size_t
instead of int to get rid of overflow.
* vm_eval.c (backtrace_object, vm_backtrace_each): ditto, use
ptrdiff_t.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@35772 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'vm.c')
-rw-r--r-- | vm.c | 16 |
1 files changed, 8 insertions, 8 deletions
@@ -822,7 +822,7 @@ frame_info_to_str_override(rb_frame_info_t *fi, VALUE *args) typedef struct rb_backtrace_struct { rb_frame_info_t *backtrace; - int backtrace_size; + size_t backtrace_size; VALUE str; } rb_backtrace_t; @@ -831,7 +831,7 @@ backtrace_mark(void *ptr) { if (ptr) { rb_backtrace_t *bt = (rb_backtrace_t *)ptr; - int i, s = bt->backtrace_size; + size_t i, s = bt->backtrace_size; for (i=0; i<s; i++) { frame_info_mark(&bt->backtrace[i]); @@ -885,19 +885,19 @@ backtrace_alloc(VALUE klass) } static VALUE -backtrace_object(rb_thread_t *th, int lev, int n) +backtrace_object(rb_thread_t *th, int lev, ptrdiff_t n) { VALUE btobj = backtrace_alloc(rb_cBacktrace); rb_backtrace_t *bt; rb_control_frame_t *last_cfp = th->cfp; rb_control_frame_t *start_cfp = RUBY_VM_END_CONTROL_FRAME(th); rb_control_frame_t *cfp; - int size, i, j; + ptrdiff_t size, i, j; start_cfp = RUBY_VM_NEXT_CONTROL_FRAME( RUBY_VM_NEXT_CONTROL_FRAME( RUBY_VM_NEXT_CONTROL_FRAME(start_cfp))); /* skip top frames */ - size = (int)(start_cfp - last_cfp) + 1; /* TODO: check overflow */ + size = (start_cfp - last_cfp) + 1; if (n <= 0) { n = size + n; @@ -949,7 +949,7 @@ static VALUE backtreace_collect(rb_backtrace_t *bt, VALUE (*func)(rb_frame_info_t *, VALUE *)) { VALUE btary; - int i; + size_t i; VALUE args[3]; rb_thread_t *th = GET_THREAD(); @@ -1016,7 +1016,7 @@ backtrace_load_data(VALUE self, VALUE str) /* old style backtrace for compatibility */ static int -vm_backtrace_each(rb_thread_t *th, int lev, int n, void (*init)(void *), rb_backtrace_iter_func *iter, void *arg) +vm_backtrace_each(rb_thread_t *th, int lev, ptrdiff_t n, void (*init)(void *), rb_backtrace_iter_func *iter, void *arg) { const rb_control_frame_t *limit_cfp = th->cfp; const rb_control_frame_t *cfp = (void *)(th->stack + th->stack_size); @@ -1024,7 +1024,7 @@ vm_backtrace_each(rb_thread_t *th, int lev, int n, void (*init)(void *), rb_back int line_no = 0; if (n <= 0) { - n = (int)(cfp - limit_cfp); /* TODO: check overflow */ + n = cfp - limit_cfp; } cfp -= 2; |