Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <1696390813.1234829399515.JavaMail.jira@brutus>
Date: Mon, 16 Feb 2009 16:09:59 -0800 (PST)
From: "Sebb (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Created: (BEANUTILS-336)
 MappedPropertyDescriptor#reLoadClass() possible NPE / odd code; also
 swallows Throwable
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

MappedPropertyDescriptor#reLoadClass() possible NPE / odd code; also swallows Throwable
---------------------------------------------------------------------------------------

                 Key: BEANUTILS-336
                 URL: https://issues.apache.org/jira/browse/BEANUTILS-336
             Project: Commons BeanUtils
          Issue Type: Bug
          Components: Bean / Property Utils
            Reporter: Sebb


MappedPropertyDescriptor#reLoadClass() possible NPE / odd code

{code}
private Class reLoadClass() {
    ClassLoader classLoader = Thread.currentThread().getContextClassLoader();
    // Try the context class loader
    if (classLoader != null) {
        try {
             return classLoader.loadClass(className);
        } catch (Throwable t) {
             // ignore
        }
    }
    // Try this class's class loader
    try {
        return classLoader.loadClass(className);
    } catch (Throwable t) {
        return null;
    }
}
{code}

The second call to classLoader.loadClass(className) is likely to fail with NPE - if it is ever called.

The comments suggest that the second loadClass() invocation should be done using a different class loader, however this is not done.

Does the code really want to try loading the class twice using the same classloader?
This is what will happen if the initial loadClass() fails.


Another problem with the code is that it swallows Throwable.
It would be better to catch just the expected Exceptions - or even Exception, but not Throwable.


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