Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(282)

Issue 10140046: code review 10140046: asn1: Improved encoding/decoding for OID types (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by gedimitr
Modified:
11 years, 9 months ago
Reviewers:
CC:
golang-dev, agl1
Visibility:
Public.

Description

asn1: Improved encoding/decoding for OID types The first identifier in an Object Identifer must be between 0 and 2 inclusive. The range of values that the second one can take depends on the value of the first one. The two first identifiers are not necessarily encoded in a single octet, but in a varint.

Patch Set 1 #

Patch Set 2 : diff -r 7eca82ec46af https://code.google.com/p/go #

Patch Set 3 : diff -r 7eca82ec46af https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -8 lines) Patch
M src/pkg/crypto/x509/x509_test.go View 1 1 chunk +1 line, -1 line 0 comments Download
M src/pkg/encoding/asn1/asn1.go View 1 1 chunk +17 lines, -5 lines 0 comments Download
M src/pkg/encoding/asn1/asn1_test.go View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/pkg/encoding/asn1/marshal.go View 1 1 chunk +2 lines, -2 lines 0 comments Download
M src/pkg/encoding/asn1/marshal_test.go View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4
gedimitr
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
11 years, 9 months ago (2013-06-10 20:52:41 UTC) #1
agl1
Why do you believe that the first two values need to be a varint? I ...
11 years, 9 months ago (2013-06-10 21:00:56 UTC) #2
gedimitr
Hi, X.690 is explicit about it: " 8.19.2 The contents octets shall be an (ordered) ...
11 years, 9 months ago (2013-06-10 21:17:21 UTC) #3
agl1
11 years, 9 months ago (2013-06-10 22:14:54 UTC) #4
*** Submitted as https://code.google.com/p/go/source/detail?r=fe447e02dc0b ***

asn1: Improved encoding/decoding for OID types

The first identifier in an Object Identifer must be between 0 and 2
inclusive. The range of values that the second one can take depends
on the value of the first one.
The two first identifiers are not necessarily encoded in a single octet,
but in a varint.

R=golang-dev, agl
CC=golang-dev
https://codereview.appspot.com/10140046

Committer: Adam Langley <agl@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b