Closed
Bug 1123374
Opened 10 years ago
Closed 10 years ago
Strange log: CertBlocklist::Init certList path: /home/mozilla/.mozilla/firefox/78bj0l2l.test5/revocations.txt....
Categories
(Core :: Security, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
Tracking | Status | |
---|---|---|
firefox36 | --- | unaffected |
firefox37 | + | fixed |
firefox38 | --- | fixed |
People
(Reporter: dragana, Assigned: keeler)
References
Details
Attachments
(1 file)
6.42 KB,
patch
|
mgoodwin
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
looking at something else, I hsave seen this strange log:
CertBlocklist::Init certList path: /home/mozilla/.mozilla/firefox/78bj0l2l.test5/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocations.txt/revocatio.....
Maybe it is just a log...
![]() |
Assignee | |
Comment 1•10 years ago
|
||
It turns out, if revocations.txt doesn't exist, we end up repeatedly appending "/revocations.txt" to the file path and seeing if that exists, which it obviously doesn't. This reorganizes the initialization logic such that this is no longer possible.
![]() |
Assignee | |
Comment 2•10 years ago
|
||
Looks like this is causing crashes: see bug 1123838.
status-firefox37:
--- → affected
status-firefox38:
--- → affected
Comment 3•10 years ago
|
||
Comment on attachment 8551917 [details] [diff] [review]
patch
Review of attachment 8551917 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me.
Attachment #8551917 -
Flags: review?(mgoodwin) → review+
![]() |
Assignee | |
Comment 4•10 years ago
|
||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
![]() |
Assignee | |
Comment 6•10 years ago
|
||
Comment on attachment 8551917 [details] [diff] [review]
patch
Approval Request Comment
[Feature/regressing bug #]: OneCRL (bug 1024809)
[User impact if declined]: OOM crashes (see bug 1123838)
[Describe test coverage new/current, TreeHerder]: no automated tests (but you can set the environment variable NSPR_LOG_MODULES to "CertBlock:5" and verify that you don't see anything like what's in comment 0)
[Risks and why]: low - this feature isn't actually being used yet
[String/UUID change made/needed]: none
Attachment #8551917 -
Flags: approval-mozilla-aurora?
![]() |
Assignee | |
Updated•10 years ago
|
status-firefox36:
--- → unaffected
Updated•10 years ago
|
Attachment #8551917 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
![]() |
Assignee | |
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 8•10 years ago
|
||
Comment 10•10 years ago
|
||
Version 20150225 is always slow and i mus use old 20141114 :-(
In Safe Mode too - how to i test what is wrong please ???
https://bugzilla.mozilla.org/show_bug.cgi?id=1134327
You need to log in
before you can comment on or make changes to this bug.
Description
•