Closed
Bug 1807008
Opened 3 years ago
Closed 3 years ago
Erroneous errors when adding multiple security devices
Categories
(Firefox :: Enterprise Policies, defect, P3)
Firefox
Enterprise Policies
Tracking
()
RESOLVED
FIXED
110 Branch
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
1.02 KB,
patch
|
RyanVM
:
approval-mozilla-esr102+
|
Details | Diff | Splinter Review |
When you add multiple security devices, you get errors even though they were added successfully.
The errors happen on every boot.
Assignee | ||
Comment 1•3 years ago
|
||
The bug is that I was storing the result of an enumerator and reusing it in a loop so on subsequent visits to the loop, we were at the end of the enumerator.
Assignee | ||
Comment 2•3 years ago
|
||
Assignee | ||
Updated•3 years ago
|
Severity: -- → S3
Priority: -- → P3
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/b346e485c27e
Use enumerator directly to enumerate security devices. r=keeler
Comment 4•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox110:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Assignee | ||
Comment 5•2 years ago
|
||
ESR patch (needed for Bug 1829197)
Assignee | ||
Comment 6•2 years ago
|
||
Comment on attachment 9332637 [details] [diff] [review]
ESR patch
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Policy only, needed for 1829197
- User impact if declined: None
- Fix Landed on Version: 110
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Policy only, has been tested in the field.
Attachment #9332637 -
Flags: approval-mozilla-esr102?
Comment 7•2 years ago
|
||
Comment on attachment 9332637 [details] [diff] [review]
ESR patch
Approved for 102.12esr.
Attachment #9332637 -
Attachment is patch: true
Attachment #9332637 -
Flags: approval-mozilla-esr102? → approval-mozilla-esr102+
Comment 8•2 years ago
|
||
bugherder uplift |
status-firefox-esr102:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•