Closed
Bug 1281612
Opened 8 years ago
Closed 8 years ago
Make memleak.sh run with NSS_ENABLE_TLS_1_3=1
Categories
(NSS :: Libraries, defect)
NSS
Libraries
Tracking
(Not tracked)
RESOLVED
FIXED
3.26
People
(Reporter: ttaubert, Assigned: ttaubert)
References
Details
Similar to bug 1263841, we should let memleak.sh ignore TLS 1.3+.
Assignee | ||
Comment 1•8 years ago
|
||
Patch at: https://codereview.appspot.com/303870043
I have two green runs on Taskcluster:
https://tools.taskcluster.net/task-inspector/#CROZh79dTCej6Iywiu5KJg/0
https://tools.taskcluster.net/task-inspector/#eEY7LQGNR4ibBifk-plHHA/0
Flags: needinfo?(martin.thomson)
Flags: needinfo?(franziskuskiefer)
Comment 2•8 years ago
|
||
r+ for the patch. But I'm still a bit concerned about simply disabling them for 1.3. I filed bug 1281707 to look into new ways of doing memleak tests to cover TLS 1.3 as well.
Flags: needinfo?(franziskuskiefer)
Assignee | ||
Comment 3•8 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.26
You need to log in
before you can comment on or make changes to this bug.
Description
•