List Info

Thread: Created: (XMLBEANS-260) Calling SchemaType.isSkippedAnonymousType() can throw a NPE




Created: (XMLBEANS-260) Calling SchemaType.isSkippedAnonymousType() can throw a NPE
user name
2006-03-29 14:43:25
Calling SchemaType.isSkippedAnonymousType() can throw a NPE
-----------------------------------------------------------

         Key: XMLBEANS-260
         URL: htt
p://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.atl
assian.com/software/jira


------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribexmlbeans.apache.org
For additional commands, e-mail: dev-helpxmlbeans.apache.org

[1]

about | contact  Other archives ( Real Estate discussion Medical topics )