Closed
Bug 1093334
Opened 10 years ago
Closed 10 years ago
Remove redundant Chromium headers from security/sandbox/linux
Categories
(Core :: Security, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: jld, Assigned: jld)
References
Details
Attachments
(3 files)
(deleted),
patch
|
kang
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
kang
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
kang
:
review+
|
Details | Diff | Splinter Review |
We have some copies of headers from Chromium, for using the Linux kernel seccomp-bpf functionality, in security/sandbox/linux. But now we have a (partial) import of the Chromium Linux sandboxing code, so we could clean things up such that the Chromium code is all in security/sandbox/chromium/linux/sandbox and only non-Chromium code is in security/sandbox/linux, which would make the file security/sandbox/linux/LICENSE unnecessary.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8532271 -
Flags: review?(gdestuynder)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8532272 -
Flags: review?(gdestuynder)
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8532273 -
Flags: review?(gdestuynder)
Assignee | ||
Comment 4•10 years ago
|
||
Comment 5•10 years ago
|
||
(In reply to Jed Davis [:jld] from comment #1)
> Created attachment 8532271 [details] [diff] [review]
> Step 1: import more Chromium headers.
Just checking ... are these from commit 9522fad406dd161400daa518075828e47bd47f60?
Flags: needinfo?(jld)
Assignee | ||
Comment 6•10 years ago
|
||
(In reply to Bob Owen (:bobowen) from comment #5)
> (In reply to Jed Davis [:jld] from comment #1)
> > Created attachment 8532271 [details] [diff] [review]
> > Step 1: import more Chromium headers.
>
> Just checking ... are these from commit
> 9522fad406dd161400daa518075828e47bd47f60?
Yes — and the hash is in the patch's commit message, for when someone runs git/hg log.
Flags: needinfo?(jld)
Attachment #8532271 -
Flags: review?(gdestuynder) → review+
Attachment #8532272 -
Flags: review?(gdestuynder) → review+
Comment on attachment 8532273 [details] [diff] [review]
Step 3: remove the old headers.
Review of attachment 8532273 [details] [diff] [review]:
-----------------------------------------------------------------
note: license still here i guess - do you plan to remove it?
Attachment #8532273 -
Flags: review?(gdestuynder) → review+
Assignee | ||
Comment 8•10 years ago
|
||
(In reply to Guillaume Destuynder [:kang] from comment #7)
> note: license still here i guess - do you plan to remove it?
Yes; thanks for catching that.
Assignee | ||
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e0f0ebdd1df5
https://hg.mozilla.org/mozilla-central/rev/1e0944ec79a6
https://hg.mozilla.org/mozilla-central/rev/0ffb57e3e9c9
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in
before you can comment on or make changes to this bug.
Description
•