SpringProperties: the ClassLoader might be null, if class is loaded by the bootstrap class loader [SPR-11721]

[spring-projects-issues]

Artem Bilan opened SPR-11721 and commented

The NPE is here:

ClassLoader cl = SpringProperties.class.getClassLoader();
			URL url = cl.getResource(PROPERTIES_RESOURCE_LOCATION);

The similar issue is here: http://stackoverflow.com/questions/1921238/getclass-getclassloader-is-null-why

---

Affects: 3.2.8, 4.0.3

Reference URL: http://stackoverflow.com/questions/23228968

Issue Links:

• PreparedStatement#getParameterMetaData() calls may trigger unwanted side effects [SPR-11386] #16013 PreparedStatement#getParameterMetaData() calls may trigger unwanted side effects
• StandardEnvironment's system environment access produces warning with stacktrace on WebSphere [SPR-11297] #15921 StandardEnvironment's system environment access produces warning with stacktrace on WebSphere
• Improve the performance of BeanInfo lookups in CachedIntrospectionResults [SPR-9014] #13653 Improve the performance of BeanInfo lookups in CachedIntrospectionResults
• PropertyEditorRegistrySupport throws ExceptionInInitializerError when on endorsed classpath [SPR-11780] #16402 PropertyEditorRegistrySupport throws ExceptionInInitializerError when on endorsed classpath

Backported to: 3.2.9

[spring-projects-issues]

Juergen Hoeller commented

Indeed, getClassLoader() may return null for the bootstrap loader depending on the JVM. We're falling back to ClassLoader.getSystemResource(name) now, since even ClassLoader.getSystemClassLoader() may return null...

Juergen

[spring-projects-issues]

Juergen Hoeller commented

While being at it, I've updated all our classpath resource accessing spots with a fallback to the system ClassLoader or (if even that is null) to getSystemResource(s).

Juergen

[spring-projects-issues]

Juergen Hoeller commented

Artem, does this work for you now, against 4.0.4 and/or the latest 3.2.9 snapshot? What does your deployment layout actually look like there?

Juergen

[spring-projects-issues]

Artem Bilan commented

Thanks, Juergen, for your attention!

Spring Integration works well with those snapshots:
https://build.spring.io/browse/INT-MEIGHT
https://build.spring.io/browse/INT-SIJDK7

I can't say anything about that issue, because the original was from SO post. I mentioned there that you fixed it, but it is without feedback