Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Tomasz Grabiec
Pekka Enberg wrote: > Tomasz Grabiec wrote: > Tomek Grabiec wrote: > > The function was setting elem to the init_stack_trace_elem_current()'s > > frame which is error prone because after function returns its stack > > frame might be overriden by another function calls done from the

Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Pekka Enberg
Tomasz Grabiec wrote: Tomek Grabiec wrote: > The function was setting elem to the init_stack_trace_elem_current()'s > frame which is error prone because after function returns its stack > frame might be overriden by another function calls done from the > caller. > > A f

Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Tomasz Grabiec
Pekka Enberg wrote: > Tomasz Grabiec wrote: > > Dnia 2009-11-17, wto o godzinie 16:39 +0200, Pekka Enberg pisze: > >> Tomek Grabiec wrote: > >>> The function was setting elem to the init_stack_trace_elem_current()'s > >>> frame which is error prone because after function returns its stack > >>> fra

Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Pekka Enberg
Tomasz Grabiec wrote: > Dnia 2009-11-17, wto o godzinie 16:39 +0200, Pekka Enberg pisze: >> Tomek Grabiec wrote: >>> The function was setting elem to the init_stack_trace_elem_current()'s >>> frame which is error prone because after function returns its stack >>> frame might be overriden by another

Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Tomasz Grabiec
Dnia 2009-11-17, wto o godzinie 16:39 +0200, Pekka Enberg pisze: > Tomek Grabiec wrote: > > The function was setting elem to the init_stack_trace_elem_current()'s > > frame which is error prone because after function returns its stack > > frame might be overriden by another function calls done from

Re: [PATCH] vm: fix init_stack_trace_elem_current()

2009-11-17 Thread Pekka Enberg
Tomek Grabiec wrote: > The function was setting elem to the init_stack_trace_elem_current()'s > frame which is error prone because after function returns its stack > frame might be overriden by another function calls done from the > caller. > > A fix for that is to make init_stack_trace_elem_curre

[PATCH] vm: fix init_stack_trace_elem_current()

2009-11-16 Thread Tomek Grabiec
The function was setting elem to the init_stack_trace_elem_current()'s frame which is error prone because after function returns its stack frame might be overriden by another function calls done from the caller. A fix for that is to make init_stack_trace_elem_current() set elem to the caller's fra