search

indutny / asn1.js

ASN.1 Decoder/Encoder/DSL
MIT License
184 stars 64 forks source link

result is not defined in decodeObjid #56

Closed TheSharpieOne closed 8 years ago

TheSharpieOne commented 8 years ago

result is never defined: https://github.com/indutny/asn1.js/blob/v4.2.1/lib/asn1/decoders/der.js#L185-L193

In the browser (asn1.js is used [eventually through dependencies] by crypto-browserify) this is causing issues.

Not sure if it was intentionally not defined or not. It was in the initial commit and I find it difficult to believe it has not come up before.

indutny commented 8 years ago

Nice catch! Do you want t submit PR for this?