Closed
Bug 1317959
Opened 8 years ago
Closed 7 years ago
Intermittent LeakSanitizer | leak at PL_ArenaAllocate, PORT_ArenaAlloc_Util, PORT_ArenaZAlloc_Util, CERT_DistNamesFromCertList
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: bulk-close-intermittents, intermittent-failure, Whiteboard: [ele:1a])
Filed by: tomcat [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=6708004&repo=autoland https://queue.taskcluster.net/v1/task/BhATAGO5SpGEeaJiOk7SOg/runs/0/artifacts/public/logs/live_backing.log
|
||
Comment 1•8 years ago
|
||
This started 1.5 days ago and has already hit beta. If it's a code regression, then it should be from this push: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=bcafa3ea224b818b4ffb165da4fa7a51f3d80280 Nils, could this be a regression from bug 1316261?
Flags: needinfo?(drno)
|
||
Comment 2•8 years ago
|
||
(In reply to Sebastian Hengst [:aryx][:archaeopteryx] (needinfo on intermittent or backout) from comment #1) > This started 1.5 days ago and has already hit beta. If it's a code > regression, then it should be from this push: > https://treeherder.mozilla.org/#/jobs?repo=mozilla- > beta&revision=bcafa3ea224b818b4ffb165da4fa7a51f3d80280 > > Nils, could this be a regression from bug 1316261? Yes that looks very plausible. Looking if/how we can fix that...
Flags: needinfo?(drno)
|
|
||
Comment 3•8 years ago
|
||
Martin it looks to me like someone needs to call CERT_FreeDistNames() on the new trust anchor we set in bug 1316261. But CERT_FreeDistNames is not exported by NSS. And I guess it would actually make more sense if NSS frees this stuff itself then the caller of SSL_SetTrustAnchors(). But that would mean this a NSS bug. What do you think?
Flags: needinfo?(martin.thomson)
| Comment hidden (Intermittent Failures Robot) |
|
||
Comment 5•8 years ago
|
||
I can confirm that this is an NSS bug. The fix is relatively simple, but it will take a while to work its way into gecko. If we're uplifting, where do you need/want to uplift? I need to know this so that I can patch NSS on the appropriate branches.
Flags: needinfo?(martin.thomson) → needinfo?(drno)
|
|
||
Comment 6•8 years ago
|
||
(In reply to Martin Thomson [:mt:] from comment #5) > I can confirm that this is an NSS bug. The fix is relatively simple, but it > will take a while to work its way into gecko. If we're uplifting, where do > you need/want to uplift? I need to know this so that I can patch NSS on the > appropriate branches. Since bug 1316261 has landed now on 52 and 53 these are the two which need to be fixed I guess. I guess in ideal case we would want to try to uplift to 51 again. And we have the request for uplift to ESR 45. But I don't know if that is feasible given that this now involves a NSS change.
Flags: needinfo?(drno)
| Comment hidden (Intermittent Failures Robot) |
|
|
||
Comment 8•8 years ago
|
||
We now need NSS 3.28 in order to land this. Since Firefox 51 uses NSS 3.27, we would need a new NSS release to land that there. Nils, do you want me to push for a new NSS release?
Depends on: 1305970
Flags: needinfo?(drno)
|
|
||
Comment 9•8 years ago
|
||
My guess would be that enterprises who are affected by bug 1316261 would be more concerned about the ESR 45 release, then simply having to wait (6 weeks ?) for the 52 rather than the 51 release. But I have no idea what it takes to update NSS on ESR. I'm assuming that is an even bigger task. If that is the case then I'm alright with shipping this in 52 as the earliest release.
Flags: needinfo?(drno)
| Comment hidden (Intermittent Failures Robot) |
Whiteboard: [ele:1a]
Status: NEW → RESOLVED
Closed: 7 years ago
Keywords: bulk-close-intermittents
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•