Closed
Bug 1308548
Opened 8 years ago
Closed 8 years ago
Deal with Bug 1296317 changing VerifyCert() and VerifySSLServerCert() to return mozilla::pkix::Result instead of SECStatus
Categories
(MailNews Core :: Security: S/MIME, defect)
MailNews Core
Security: S/MIME
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 52.0
People
(Reporter: Cykesiopka, Unassigned)
References
Details
Attachments
(1 file)
|
6.44 KB,
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
Bug 1296317 will change VerifyCert() and VerifySSLServerCert() to return mozilla::pkix::Result instead of SECStatus. c-c code will need to be updated to deal with this.
|
Reporter | |
Comment 1•8 years ago
|
||
This compiles on my machine for TB, but I haven't tested that the features still work. If it doesn't work, then at least this patch is a starting point.
Attachment #8798943 -
Flags: review?(rkent)
|
||
Comment 3•8 years ago
|
||
Comment on attachment 8798943 [details] [diff] [review] bug1308548_VerifyCert-VerifySSLServerCert-port-switch-to-pkix-Result_v1.patch Review of attachment 8798943 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, much appreciated!
Attachment #8798943 -
Flags: review?(rkent) → review+
|
|
||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/comm-central/rev/6d463070d845ed47088ee322e0bca4dc4bf12ae3 We'll reopen if something doesn't work (which I doubt). Thanks again!!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
You need to log in
before you can comment on or make changes to this bug.
Description
•