Closed Bug 1281707 Opened 8 years ago Closed 8 years ago

New memleak tests

Categories

(NSS :: Test, defect, P2)

defect

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.
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?
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.
I agree with Tim regarding [AL]San.
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.