[prev in list] [next in list] [prev in thread] [next in thread]
List: xmlbeans-dev
Subject: [jira] Created: (XMLBEANS-260) Calling
From: "Luke Peters (JIRA)" <xmlbeans-dev () xml ! apache ! org>
Date: 2006-03-29 14:43:25
Message-ID: 378864284.1143643405072.JavaMail.jira () ajax
[Download RAW message or body]
Calling SchemaType.isSkippedAnonymousType() can throw a NPE
-----------------------------------------------------------
Key: XMLBEANS-260
URL: http://issues.apache.org/jira/browse/XMLBEANS-260
Project: XMLBeans
Type: Bug
Components: XmlObject
Versions: Version 2.1
Environment: Java 1.5, Windows XP
Reporter: Luke Peters
Fix For: Version 2.1
If isSkippedAnonymousType() is called on a SchemaType instance that does not have \
_outerSchemaTypeRef set, then a NPE will be thrown. Example:
MyXmlBean myBean = MyXmlBean.Factory.newInstance();
boolean isSkipped = myBean.schemaType().isSkippedAnonymousType(); <- throws a NPE
It seems that it should return false in this instance, rather than throw an \
exception. All this needs to fix is to add a null check. I'm not sure which versions \
exactly are effected, but it is an issue in 2.1.
Current code:
public boolean isSkippedAnonymousType()
{ return _outerSchemaTypeRef.get().getBaseType() == this ||
_outerSchemaTypeRef.get().getContentBasedOnType() == this; }
Potential fix:
public boolean isSkippedAnonymousType()
{ return _outerSchemaTypeRef == null ? false : \
_outerSchemaTypeRef.get().getBaseType() == this ||
_outerSchemaTypeRef.get().getContentBasedOnType() == this; }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xmlbeans.apache.org
For additional commands, e-mail: dev-help@xmlbeans.apache.org
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic