More accepts() get NPE guards

Issue: http://jira.grails.org/browse/GRAILS-9651
This commit is contained in:
Andy Clement
2012-12-10 08:25:08 -08:00
parent dcee3af4be
commit 2fe68af988
3 changed files with 9 additions and 1 deletions

View File

@@ -32,6 +32,9 @@ public class CglibPlugin implements LoadtimeInstrumentationPlugin {
// implementing LoadtimeInstrumentationPlugin
public boolean accept(String slashedTypeName, ClassLoader classLoader, ProtectionDomain protectionDomain, byte[] bytes) {
if (slashedTypeName==null) {
return false;
}
return slashedTypeName.equals("net/sf/cglib/core/AbstractClassGenerator");
// || slashedTypeName.equals("net/sf/cglib/reflect/FastClass");
}

View File

@@ -51,6 +51,9 @@ public class GroovyPlugin implements LoadtimeInstrumentationPlugin, ReloadEventP
// implementing LoadtimeInstrumentationPlugin
public boolean accept(String slashedTypeName, ClassLoader classLoader, ProtectionDomain protectionDomain, byte[] bytes) {
// TODO take classloader into account?
if (slashedTypeName==null) {
return false;
}
if (!allowCompilableCallSites) {
return slashedTypeName.equals("org/codehaus/groovy/runtime/callsite/GroovySunClassLoader")
|| slashedTypeName.equals("org/codehaus/groovy/runtime/callsite/CallSiteGenerator");

View File

@@ -63,7 +63,9 @@ public class SpringPlugin implements LoadtimeInstrumentationPlugin, ReloadEventP
public boolean accept(String slashedTypeName, ClassLoader classLoader, ProtectionDomain protectionDomain, byte[] bytes) {
// TODO take classloader into account?
if (slashedTypeName == null) {
return false;
}
// Just interested in whether this type got loaded
if (slashedTypeName.equals("org/springframework/beans/CachedIntrospectionResults")) {
cachedIntrospectionResultsClassLoaded = true;