ssl.def should have SSL_SetDowngradeCheckVersion

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: emk, Unassigned)

Tracking

trunk

Firefox Tracking Flags

(firefox47 affected)

Details

Attachments

(1 attachment)

Comment hidden (empty)

Comment 1

3 years ago
Oops. Yes. Patch welcome :)
(Reporter)

Comment 2

3 years ago
Created attachment 8719016 [details] [diff] [review]
Add SSL_SetDowngradeCheckVersion to ssl.def

A Linux build passed with the patch.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=81e17edc1425
Attachment #8719016 - Flags: review?(ekr)

Comment 3

3 years ago
Comment on attachment 8719016 [details] [diff] [review]
Add SSL_SetDowngradeCheckVersion to ssl.def

LGTM
Attachment #8719016 - Flags: review?(ekr) → review+
(Reporter)

Comment 4

3 years ago
I do not have a commit access to the NSS repo. Could you land the patch?
Keywords: checkin-needed
(Reporter)

Comment 6

3 years ago
Thanks!
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.