From: brian <brian@brianweb.net>
Date: Thu, 1 May 2008 17:01:17 +0000 (+0000)
Subject: fix bug with unequally sized certs
X-Git-Url: http://git.megacz.com/?a=commitdiff_plain;h=60a2432eb194b4a93d2953f7781384e64f5e414d;p=org.ibex.crypto.git

fix bug with unequally sized certs

darcs-hash:20080501170117-24bed-f8b6c5be8423864e944a658a74f6b0a4d97c4d95.gz
---

diff --git a/src/org/ibex/net/SSL.java b/src/org/ibex/net/SSL.java
index 0f15b28..dc1c201 100644
--- a/src/org/ibex/net/SSL.java
+++ b/src/org/ibex/net/SSL.java
@@ -326,12 +326,18 @@ public class SSL extends Socket {
         byte[] buf = readHandshake();
         if(buf[0] != 11) throw new Exn("expected a Certificate message");
         if((((buf[4]&0xff)<<16)|((buf[5]&0xff)<<8)|((buf[6]&0xff)<<0)) != buf.length-7) throw new Exn("size mismatch in Certificate message");
-        int p = 7;
-        int count = 0;
+        int p;
+        int count;
         
-        for(int i=p;i<buf.length-3;i+=((buf[p+0]&0xff)<<16)|((buf[p+1]&0xff)<<8)|((buf[p+2]&0xff)<<0)) count++;
+        p = 7;
+        count = 0;
+        while(p < buf.length - 3) {
+            p += 3 + ((buf[p+0]&0xff)<<16)|((buf[p+1]&0xff)<<8)|((buf[p+2]&0xff)<<0);
+            count++;
+        }
         if(count == 0) throw new Exn("server didn't provide any certificates");
         X509.Certificate[] certs = new X509.Certificate[count];
+        p = 7;
         count = 0;
         while(p < buf.length) {
             int len = ((buf[p+0]&0xff)<<16)|((buf[p+1]&0xff)<<8)|((buf[p+2]&0xff)<<0);