[jboss-remoting-commits] JBoss Remoting SVN: r4702 - remoting3/trunk/transporter/src/main/java/org/jboss/remoting/transporter.

[jboss-remoting-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2008-11-19 09:41:04 -0500 (Wed, 19 Nov 2008)
New Revision: 4702

Modified:
   remoting3/trunk/transporter/src/main/java/org/jboss/remoting/transporter/TransporterInvocation.java
Log:
Fix a bug where fields are not updated; also make sure the length value is read as unsigned short

Modified: remoting3/trunk/transporter/src/main/java/org/jboss/remoting/transporter/TransporterInvocation.java
===================================================================
--- remoting3/trunk/transporter/src/main/java/org/jboss/remoting/transporter/TransporterInvocation.java	2008-11-18 20:39:56 UTC (rev 4701)
+++ remoting3/trunk/transporter/src/main/java/org/jboss/remoting/transporter/TransporterInvocation.java	2008-11-19 14:41:04 UTC (rev 4702)
@@ -68,7 +68,7 @@
 
     public void readExternal(final ObjectInput in) throws IOException, ClassNotFoundException {
         name = (String) in.readObject();
-        final short cnt = in.readShort();
+        final int cnt = in.readShort() & 0xffff;
         final Class<?>[] parameterTypes = new Class<?>[cnt];
         for (short i = 0; i < cnt; i ++) {
             parameterTypes[i] = (Class<?>) in.readObject();
@@ -77,6 +77,8 @@
         for (short i = 0; i < cnt; i ++) {
             args[i] = in.readObject();
         }
+        this.parameterTypes = parameterTypes;
+        this.args = args;
     }
 
     public String getName() {