2009/9/5 Pekka Enberg <penb...@cs.helsinki.fi>:
> On Sat, 2009-09-05 at 16:06 +0200, Vegard Nossum wrote:
>> Um, wait, what's the actual dereference that causes the valgrind
>> warning? Because as it stands, it looks like you're fixing just the
>> symptom and not the real error. We have method initializers that
>> should take care of initializing everything, field by field. I'd think
>> that's where you'd fix it.
>
> As you can see in the valgrind trace, the problem comes from:
>
>  cu->stack_frame = alloc_stack_frame(
>                        method->args_count,
>                        method->code_attribute.max_locals);
>
> which means ->code_attribute.max_locals is not initialized. Looking at
> cafebabe, I don't see any guarantees that everything will be
> initialized.

That's probably because there is no code attribute to begin with, in
which case we should not be preparing the method for jit..?


Vegard

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Jatovm-devel mailing list
Jatovm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jatovm-devel

Reply via email to