CXF assumes class.getPackage() is never null
--------------------------------------------

                 Key: CXF-1689
                 URL: https://issues.apache.org/jira/browse/CXF-1689
             Project: CXF
          Issue Type: Bug
          Components: Core
    Affects Versions: 2.1.1
            Reporter: Hani Suleiman
             Fix For: 2.1.2, 2.0.8


One example, JAXBExtensionHelper has this line:

cls.getPackage().getAnnotation(XmlSchema.class)

According to the javadocs of Class.getPackage, a classloader may return null if 
it has not defined a package for the class. Specifying a package is optional, 
so the class needs to handle getPackage() returning null.

Note that some CXF developers seem to understand this restriction, and so parts 
of the codebase perform the correct null check for getPackage(), but others 
seem unaware of it, and so do not check for null. The right solution is to go 
through every call to getPackage() and add a null check.

If needed (and if it'd speed up the fixing), I can supply a patch to apply to 
fix all occurrences of this issue.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to