fixed asn1_oid_to_string() conversion

diff --git a/src/libstrongswan/asn1/asn1.c b/src/libstrongswan/asn1/asn1.c
index f80c2b93bad6d8d429ed9fec9efa6af24d3be538..b0fd7f8003a6af6cf8f1655876ef3d4339155d1a 100644 (file)
--- a/src/libstrongswan/asn1/asn1.c
+++ b/src/libstrongswan/asn1/asn1.c
@@ -183,36 +183,30 @@ char *asn1_oid_to_string(chunk_t oid)
                return NULL;
        }
        val = oid.ptr[0] / 40;
-       len = snprintf(buf, sizeof(buf), "%d.%d", val, oid.ptr[0] - val * 40);
+       len = snprintf(buf, sizeof(buf), "%u.%u", val, oid.ptr[0] - val * 40);
        oid = chunk_skip(oid, 1);
        if (len < 0 || len >= sizeof(buf))
        {
                return NULL;
        }
        pos += len;
+       val = 0;
 
        while (oid.len)
        {
+               val = (val << 7) + (u_int)(oid.ptr[0] & 0x7f);
+
                if (oid.ptr[0] < 128)
                {
-                       len = snprintf(pos, sizeof(buf) + buf - pos, ".%d", oid.ptr[0]);
-                       oid = chunk_skip(oid, 1);
-               }
-               else
-               {
-                       if (oid.len == 1)
+                       len = snprintf(pos, sizeof(buf) + buf - pos, ".%u", val);
+                       if (len < 0 || len >= sizeof(buf) + buf - pos)
                        {
                                return NULL;
                        }
-                       val = ((u_int)(oid.ptr[0] & 0x7F) << 7) + oid.ptr[1];
-                       len = snprintf(pos, sizeof(buf) + buf - pos, ".%d", val);
-                       oid = chunk_skip(oid, 2);
-               }
-               if (len < 0 || len >= sizeof(buf) + buf - pos)
-               {
-                       return NULL;
+                       pos += len;
+                       val = 0;
                }
-               pos += len;
+               oid = chunk_skip(oid, 1);
        }
        return strdup(buf);
 }