[bug-classpath] [Bug classpath/22998] New: Null check after dereference in gnu.xml.aelfred2.XmlParser

[pinskia at gcc dot gnu dot org]

In classpath-0.15:



In gnu.xml.aelfred2.XmlParser, at line 1377, is the following code:



       if (type.equals("CDATA") || type == null)



Obviously, the null check cannot take place, since a null

pointer exception would already have been be thrown if type

had been null.



Found by FindBugs, http://findbugs.sourceforge.net


------- Additional Comments From pinskia at gcc dot gnu dot org  2005-07-26 
16:06 -------
Fixed by:
2005-06-21  Lillian Angel  <address@hidden>

        * gnu/xml/aelfred2/XmlParser
        (parseAttribute): if (type.equals("CDATA") || type == null)
        this would always throw a NullPointerException if type is null.

-- 
           Summary: Null check after dereference in
                    gnu.xml.aelfred2.XmlParser
           Product: classpath
           Version: unspecified
            Status: RESOLVED
          Severity: normal
          Priority: P3
         Component: classpath
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: from-classpath at savannah dot gnu dot org
                CC: bug-classpath at gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22998