Closed
Bug 997370
Opened 10 years ago
Closed 10 years ago
nsIX509Cert idl incorrect documentation of sha1fingerprint and md5fingerprint
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
mozilla33
People
(Reporter: cviecco, Assigned: hpathak)
References
Details
(Whiteboard: [good next bug][mentor=cviecco])
Attachments
(1 file, 2 obsolete files)
1.01 KB,
patch
|
hpathak
:
review+
|
Details | Diff | Splinter Review |
While helping write some heartbleed bugs it was discovered that the idl documentation in nsIX509Cert.idl for the sha1Fingerprint and md5Fingerprint do not match the impleeentation. The documentation says: The fingerprint of the certificate's public key, calculated using the $CHOICE algorithm and it should be The fingerprint of the certificate's DER encoding, calculated using the $CHOICE algorithm
Updated•10 years ago
|
Whiteboard: [good first bug]
Comment 2•10 years ago
|
||
Switching the whiteboard tag to [good next bug] . Even if it's docs, nothing that touches crypto gets to be a good first bug. Camilo, would you be willing to mentor a contributor through this?
Flags: needinfo?(cviecco)
Whiteboard: [good first bug] → [good next bug]
Updated•10 years ago
|
Whiteboard: [good next bug] → [good next bug][mentor=cviecco]
Attachment #8435946 -
Flags: review?(cviecco)
Reporter | ||
Comment 5•10 years ago
|
||
Comment on attachment 8435946 [details] [diff] [review] Updated comment. Review of attachment 8435946 [details] [diff] [review]: ----------------------------------------------------------------- very good. r- for not compiling (should be an easy fix). For pushing to try see https://wiki.mozilla.org/ReleaseEngineering/TryServer#Try_Server ::: security/manager/ssl/src/nsNSSCertificateDB.cpp @@ +1,1 @@ > +SetCertTrust/* This Source Code Form is subject to the terms of the Mozilla Public Opps, this will not compile. Do a try run to see it :)
Attachment #8435946 -
Flags: review?(cviecco) → review-
Comment on attachment 8435946 [details] [diff] [review] Updated comment. Review of attachment 8435946 [details] [diff] [review]: ----------------------------------------------------------------- Also, patch formatting should have 8 lines of unified context - see the section on writing a ~/.hgrc file here: https://developer.mozilla.org/en-US/docs/Mercurial_FAQ Finally, the patch comment should have "bug 997370 - " at the beginning. When you get an r+, you'll add r=<the reviewer's irc handle> at the end.
https://tbpl.mozilla.org/?tree=Try&rev=5113b5eabf97. Hope this goes through.
Attachment #8435946 -
Attachment is obsolete: true
Attachment #8437016 -
Flags: review?(cviecco)
Reporter | ||
Updated•10 years ago
|
Attachment #8437016 -
Flags: review?(cviecco) → review+
Received r+ from Camilo.
Attachment #8437016 -
Attachment is obsolete: true
Attachment #8437070 -
Flags: review+
Attachment #8437070 -
Flags: checkin?(cviecco)
Comment 9•10 years ago
|
||
Comment on attachment 8437070 [details] [diff] [review] Bug997370_comment_fix.diff In the future, you can just use the checkin-needed bug keyword :)
Attachment #8437070 -
Flags: checkin?(cviecco)
Comment 11•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4f9febf724b0
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in
before you can comment on or make changes to this bug.
Description
•