Closed
Bug 252271
Opened 20 years ago
Closed 20 years ago
CERT_VerifyCert with param certUsageVerifyCA fails or asserts
Categories
(NSS :: Libraries, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
3.9.3
People
(Reporter: KaiE, Assigned: nelson)
References
Details
Attachments
(1 file)
3.46 KB,
patch
|
rrelyea
:
review+
julien.pierre
:
superreview+
|
Details | Diff | Splinter Review |
Bob recommended I could call CERT_VerifyCert with certUsageVerifyCA to check whether the given cert is a valid CA cert. However, this crashes using NSS code on branch MOZILLA_1_7_0_BRANCH as of Mozilla 1.7.1
Reporter | ||
Comment 1•20 years ago
|
||
If you agree with this patch, I would like to land it on MOZILLA_1_7_0_BRANCH as part of the fix for bug 249004.
Comment 2•20 years ago
|
||
Comment on attachment 153769 [details] [diff] [review] Proposed fix I'd like Julien to review this as well. If it goes in, I'd like to see it go into the trunk and 3.9 branch as well. bob
Attachment #153769 -
Flags: superreview?(julien.pierre.bugs)
Attachment #153769 -
Flags: review+
Comment 3•20 years ago
|
||
Kai, Was this case just asserting in the "default" case of the switch statement, or is it also crashing in optimized code, and if so, how ? I'd like to understand this better before putting the sr+ flag.
Reporter | ||
Comment 4•20 years ago
|
||
Julien, yes, it was crashing because of the PORT_Assert in the default statement.
Updated•20 years ago
|
Attachment #153769 -
Flags: superreview?(julien.pierre.bugs) → superreview+
Updated•20 years ago
|
Summary: CERT_VerifyCert with param certUsageVerifyCA crashes → CERT_VerifyCert with param certUsageVerifyCA fails or asserts
Reporter | ||
Comment 5•20 years ago
|
||
Could you please include this patch in NSS_CLIENT_TAG? This is required to land bug 249004 on the trunk of Mozilla. Thanks a lot.
Reporter | ||
Updated•20 years ago
|
Assignee | ||
Comment 6•20 years ago
|
||
Marking P1 for NSS 3.9.3. Note that this bug (midssing case) was fixed on the trunk in rev 1.65, back in April. That revision was supposently ported to the NSS 3.9 branch for mozilla 1.7, but this fix was apprently omitted. :( So, this patch needs to go into the 3.9 branch, and the tag moved to the new revision for that file only.
Priority: -- → P1
Target Milestone: --- → 3.9.3
Assignee | ||
Comment 7•20 years ago
|
||
Taking bug.
Assignee: wchang0222 → nelson
Hardware: PC → All
Version: unspecified → 3.9
Assignee | ||
Comment 8•20 years ago
|
||
Checking in certdb.c; new revision: 1.61.2.4; previous revision: 1.61.2.3 I moved the NSS_CLIENT_TAG from 1.61.2.3 to 1.61.2.4
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•