From b731f7ad15478d3ec463c5c1ce07a2935ce8c10d Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Wed, 17 Jun 2015 16:54:56 +0200 Subject: [PATCH] ObjenesisCglibAopProxy catches NoClassDefFoundError as well Issue: SPR-13131 --- .../aop/framework/ObjenesisCglibAopProxy.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/ObjenesisCglibAopProxy.java b/spring-aop/src/main/java/org/springframework/aop/framework/ObjenesisCglibAopProxy.java index d359e9ff2a..6b7d06aeb9 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/ObjenesisCglibAopProxy.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/ObjenesisCglibAopProxy.java @@ -22,7 +22,6 @@ import org.apache.commons.logging.LogFactory; import org.springframework.cglib.proxy.Callback; import org.springframework.cglib.proxy.Enhancer; import org.springframework.cglib.proxy.Factory; -import org.springframework.objenesis.ObjenesisException; import org.springframework.objenesis.ObjenesisStd; /** @@ -60,7 +59,7 @@ class ObjenesisCglibAopProxy extends CglibAopProxy { try { proxyInstance = this.objenesis.newInstance(proxyClass); } - catch (ObjenesisException ex) { + catch (Throwable ex) { logger.debug("Unable to instantiate proxy using Objenesis, " + "falling back to regular proxy construction", ex); } @@ -72,7 +71,7 @@ class ObjenesisCglibAopProxy extends CglibAopProxy { proxyClass.getConstructor(this.constructorArgTypes).newInstance(this.constructorArgs) : proxyClass.newInstance()); } - catch (Exception ex) { + catch (Throwable ex) { throw new AopConfigException("Unable to instantiate proxy using Objenesis, " + "and regular proxy instantiation via default constructor fails as well", ex); }