Hi,
I'm experiencing an exception under heavy load using hibernate, which I believe
is a javassist bug.
The code that ProxyFactory.makeForwarder generates to invoke the handler is:
| * if (methods[index * 2] == null) {
| * methods[index * 2]
| * = RuntimeSupport.findMethod(this, <overridden name>, <desc>);
| * methods[index * 2 + 1]
| * = RuntimeSupport.findMethod(this, <delegator name>, <desc>);
| * or = null // the original method is abstract.
| * }
| * return ($r)handler.invoke(this, methods[index * 2],
| * methods[index * 2 + 1], $args);
|
|
This is problematic, because two threads could the same proxy in close enough
succession that handler.invoke() could be called with a null third argument:
Thread a: methods[index *2] = ...;
Thread b: if (methods[index * 2] == null) //will evaluate to false
Thread b: loads a null methods[index *2 + 1], invokes handler
Thread a: methods[index *2 + 1] = ...;
and we get a NPE along the lines of:
| java.lang.NullPointerException
| at
javassist.util.proxy.RuntimeSupport$DefaultMethodHandler.invoke(RuntimeSupport.java:37)
| at
edu.academyart.model.Post_$$_javassist_2.getTextTitles(Post_$$_javassist_2.java)
|
I think this can be fixed by simply reversing the order that the methods are
stored in the array, which ensures both exist if the if() block is skipped, as
in the below patch.
| $ cvs diff -c2p src/main/javassist/util/proxy/ProxyFactory.java
| Index: src/main/javassist/util/proxy/ProxyFactory.java
| ===================================================================
| RCS file:
/cvsroot/jboss/javassist/src/main/javassist/util/proxy/ProxyFactory.java,v
| retrieving revision 1.26
| diff -c -2 -p -r1.26 ProxyFactory.java
| *** src/main/javassist/util/proxy/ProxyFactory.java 2 Oct 2007 03:49:11
-0000 1.26
| --- src/main/javassist/util/proxy/ProxyFactory.java 27 Dec 2007
02:47:28 -0000
| *************** public class ProxyFactory {
| *** 846,851 ****
| code.addIndex(0);
|
| - callFindMethod(code, "findSuperMethod", arrayVar, origIndex,
meth.getName(), desc);
| callFindMethod(code, "findMethod", arrayVar, delIndex,
delegatorName, desc);
|
| int pc2 = code.currentPc();
| --- 846,851 ----
| code.addIndex(0);
|
| callFindMethod(code, "findMethod", arrayVar, delIndex,
delegatorName, desc);
| + callFindMethod(code, "findSuperMethod", arrayVar, origIndex,
meth.getName(), desc);
|
| int pc2 = code.currentPc();
|
|
Please let me know if this seems reasonable, and I'll open a JIRA issue with a
patch.
View the original post :
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4115630#4115630
Reply to the post :
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4115630
_______________________________________________
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user
