geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vamsavardhana Reddy" <c1vams...@gmail.com>
Subject Re: NPE in GBeanOverride
Date Tue, 05 Feb 2008 16:08:35 GMT
Go ahead and create a JIRA.

++Vamsi

On Feb 5, 2008 9:08 PM, Vasily Zakharov <vmz@apache.org> wrote:

> Hi, all,
>
> It seems I've found a flaw in
> org.apache.geronimo.system.configuration.GBeanOverride:
>
> getValue(), lines 388-389:
> PropertyEditor editor = loadPropertyEditor(attribute, classLoader);
> editor.setAsText(value);
>
> loadPropertyEditor() may return null (lines 402, 407) and this can cause
> NPE.
>
> I didn't see this NPE on Sun, but it occurs on Harmony (clearly some
> other issue exists causing the loadPropertyEditor() to return null,
> investigating) and I think it's a problem anyway.
>
> I'm not sure what should be an appropriate behavior in case of editor is
> null.
>
> Should I file a JIRA for this?
>
> Thanks,
> Vasily
>

Mime
View raw message