[Mono-bugs] [Bug 76012][Nor] Changed - X509Certificates.cs: Parse Problem Of Version

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by sebastien at ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=76012

--- shadow/76012	2005-09-07 12:41:22.000000000 -0400
+++ shadow/76012.tmp.27518	2005-09-07 13:36:41.000000000 -0400
@@ -1,15 +1,15 @@
 Bug#: 76012
 Product: Mono: Class Libraries
 Version: 1.1
 OS: All
 OS Details: 
-Status: NEW   
+Status: ASSIGNED   
 Resolution: 
-Severity: 
-Priority: Wishlist
+Severity: Unknown
+Priority: Normal
 Component: Mono.Security
 AssignedTo: sebastien at ximian.com                            
 ReportedBy: sascha.kiefer at dialogika.de               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
@@ -42,6 +42,17 @@
    version += vers.Value [0];	// zero based
    tbs++;
 }
 
 HTH
 Sascha
+
+------- Additional Comments From sebastien at ximian.com  2005-09-07 13:36 -------
+You're right. The current code is adding the tag the element inside
+0xA0 - which is 0x02 for an integer. So the current code works for v1
+and v3 certificates (and I didn't have any v2 certificate to test with).
+
+I'll apply a similar patch (no need to create a new ASN1 as the parser
+is recusrive on constructed types) after 1.1.9 is released (very soon
+now).
+
+Thanks for spotting this!