Vegard Nossum wrote:
> 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..?

Good point. I wonder why we have methods without code attribute in 
classfiles.

------------------------------------------------------------------------------
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