X-Git-Url: http://git.indexdata.com/?a=blobdiff_plain;f=odr%2Fber_any.c;h=35c0a83302da5f1526a56a44a5e9e1e235693518;hb=04cde6d037338d3128c77698d4c8a7f03010db1e;hp=8c661f0655b1dec35310c11a6c3d797ebb9010ce;hpb=10981b2f116449523a9f628c20a78212ce91553e;p=yaz-moved-to-github.git

diff --git a/odr/ber_any.c b/odr/ber_any.c
index 8c661f0..35c0a83 100644
--- a/odr/ber_any.c
+++ b/odr/ber_any.c
@@ -4,7 +4,19 @@
  * Sebastian Hammer, Adam Dickmeiss
  *
  * $Log: ber_any.c,v $
- * Revision 1.1  1995-02-09 15:51:45  quinn
+ * Revision 1.5  1995-02-14 20:39:54  quinn
+ * Fixed bugs in completeBER and (serious one in) ber_oid.
+ *
+ * Revision 1.4  1995/02/14  11:54:33  quinn
+ * Adjustments.
+ *
+ * Revision 1.3  1995/02/10  18:57:24  quinn
+ * More in the way of error-checking.
+ *
+ * Revision 1.2  1995/02/10  15:55:28  quinn
+ * Bug fixes, mostly.
+ *
+ * Revision 1.1  1995/02/09  15:51:45  quinn
  * Works better now.
  *
  */
@@ -18,7 +30,7 @@ int ber_any(ODR o, Odr_any **p)
     switch (o->direction)
     {
     	case ODR_DECODE:
-	    if ((res = completeBER(o->bp, o->left)) <= 0)
+	    if ((res = completeBER(o->bp, 1000)) <= 0)        /* FIX THIS */
 	    	return 0;
 	    (*p)->buf = nalloc(o, res);
 	    memcpy((*p)->buf, o->bp, res);
@@ -38,40 +50,42 @@ int ber_any(ODR o, Odr_any **p)
 }
 
 /*
- * Return length of BER-package or -1.
+ * Return length of BER-package or 0.
  */
 int completeBER(unsigned char *buf, int len)
 {
     int res, ll, class, tag, cons;
     unsigned char *b = buf;
     
+    if (!len)
+    	return 0;
     if (!buf[0] && !buf[1])
-    	return -1;
+    	return 0;
     if ((res = ber_dectag(b, &class, &tag, &cons)) <= 0)
     	return 0;
     if (res > len)
-    	return -1;
+    	return 0;
     b += res;
     len -= res;
     if ((res = ber_declen(b, &ll)) <= 0)
-    	return -1;
+    	return 0;
     if (res > len)
-    	return -1;
+    	return 0;
     b += res;
     len -= res;
     if (ll >= 0)
-    	return (len >= ll ? len + (b-buf) : -1);
+    	return (len >= ll ? ll + (b-buf) : 0);
     if (!cons)
-    	return -1;    
-    while (1)
+    	return 0;    
+    /* constructed - cycle through children */
+    while (len >= 2)
     {
-	if ((res = completeBER(b, len)) < 0)
-	    return -1;
+	if (*b == 0 && *(b + 1) == 0)
+	    break;
+	if (!(res = completeBER(b, len)))
+	    return 0;
 	b += res;
 	len -= res;
-	if (len < 2)
-	    return -1;
-	if (*b == 0 && *(b + 1) == 0)
-	    return (b - buf) + 2;
     }
+    return (b - buf) + 2;
 }