Closed
Bug 1198481
Opened 9 years ago
Closed 7 years ago
fix id_pk_serverAuth typo in pkixcert.cpp
Categories
(Core :: Security: PSM, defect, P1)
Core
Security: PSM
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: keeler, Assigned: manikishanghantasala)
Details
(Whiteboard: [psm-assigned])
Attachments
(1 file, 1 obsolete file)
1.48 KB,
patch
|
keeler
:
review+
|
Details | Diff | Splinter Review |
See https://dxr.mozilla.org/mozilla-central/rev/04b8c412d9f58fb6194c58dcaa66bf278bbd53cf/security/pkix/lib/pkixcert.cpp?offset=200#160 The "id_pk_serverAuth" should be "id_kp_serverAuth".
Reporter | ||
Updated•8 years ago
|
Whiteboard: [psm-cleanup]
Reporter | ||
Updated•7 years ago
|
Priority: -- → P5
Assignee | ||
Comment 1•7 years ago
|
||
I would like to take this over . I am a newbie?
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to manikishanghantasala from comment #1) > I would like to take this over . I am a newbie.
Assignee | ||
Comment 3•7 years ago
|
||
[Security approval request comment] How easily could an exploit be constructed based on the patch? Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem? Which older supported branches are affected by this flaw? If not all supported branches, which bug introduced the flaw? Do you have backports for the affected branches? If not, how different, hard to create, and risky will they be? How likely is this patch to cause regressions; how much testing does it need?
Attachment #8933903 -
Flags: sec-approval?
Attachment #8933903 -
Flags: review+
Comment 4•7 years ago
|
||
Comment on attachment 8933903 [details]
bug-1198481
Settings sec-approval? on an unrated, public, non-security bug without answering any of the template questions isn't very useful.
Attachment #8933903 -
Flags: sec-approval?
Assignee | ||
Comment 5•7 years ago
|
||
sorry I am new to this can you please help me explaining what should be done?
Attachment #8933903 -
Flags: review+
Comment 6•7 years ago
|
||
(In reply to manikishanghantasala from comment #5) > sorry I am new to this can you please help me explaining what should be done? Please don't change any further bug flags :) The patch you uploaded is empty. When you have a new version ready, please upload it as type "patch" and ask for review? from :keeler. Then we can assign this one to you.
Assignee | ||
Comment 7•7 years ago
|
||
Attachment #8934115 -
Flags: review?(dkeeler)
Updated•7 years ago
|
Attachment #8933903 -
Attachment is obsolete: true
Updated•7 years ago
|
Assignee: nobody → manikishanghantasala
Status: NEW → ASSIGNED
Reporter | ||
Comment 8•7 years ago
|
||
Comment on attachment 8934115 [details] [diff] [review] Bug-1198481.patch Review of attachment 8934115 [details] [diff] [review]: ----------------------------------------------------------------- Great - thanks! Normally the next step would be to run this through try, but since this is a comment-only change, we can mark it `checkin-needed` directly.
Attachment #8934115 -
Flags: review?(dkeeler) → review+
Reporter | ||
Updated•7 years ago
|
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8577d0b83990 Fixed typo 'id_pk_serverAuth' to 'id_kp_serverAuth'. r=keeler
Keywords: checkin-needed
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8577d0b83990
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Updated•6 years ago
|
status-firefox43:
affected → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•