Done. http://issues.apache.org/jira/browse/GERONIMO-3814 "NPE in GBeanOverride"
Thanks! Vasily On Feb 5, 2008 7:08 PM, Vamsavardhana Reddy <[EMAIL PROTECTED]> wrote: > Go ahead and create a JIRA. > > ++Vamsi > > > > On Feb 5, 2008 9:08 PM, Vasily Zakharov <[EMAIL PROTECTED]> 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 > > > >
