Closed
Bug 1281707
Opened 8 years ago
Closed 8 years ago
New memleak tests
Categories
(NSS :: Test, defect, P2)
NSS
Test
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1246801
People
(Reporter: franziskus, Unassigned)
References
Details
When disabling memleak tests for TLS 1.3 in bug 1281612 we need something else with similar properties for these new code paths.
Comment 1•8 years ago
|
||
Having special memleak test suites sounds wrong to me, I think running everything we have with ASan builds is the way to go. I don't know enough about how the current memleak tests work but I suspect that we could get the same coverage with LSan.
So how about just duping this to bug 1246801 and focusing on that?
Reporter | ||
Comment 2•8 years ago
|
||
Sure, if LSan gives us the coverage we need, that's fine with me. That might allow us to remove the old memleak tests all together. I just want to make sure we have something running.
Comment 3•8 years ago
|
||
I agree with Tim regarding [AL]San.
Comment 4•8 years ago
|
||
Ok, let's dupe this to bug 1246801. I'll take a look.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•