Closed Bug 1144514 Opened 10 years ago Closed 10 years ago

[gonk-lmr1] Whitelist pread64

Categories

(Core :: Security: Process Sandboxing, defect)

Product:
Core
Component:
Security: Process Sandboxing
Platform:
All
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla39
Project Flags:
blocking-b2g 2.2+
Tracking Flags:
Tracking Status
firefox37 --- wontfix
firefox38 --- wontfix
firefox39 --- fixed
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: m1, Assigned: jld)

References

Details

(Whiteboard: [caf priority: p2][CR 810045])

Attachments

(1 file)

Patch
(deleted), patch
kang
: review+
bajaj
: approval-mozilla-b2g37+
Details | Diff | Splinter Review
The bionic linker uses pread64() [1] instead of read() [2] in L MR 1. security/sandbox/linux/SandboxFilter.cpp needs to be updated to whitelist pread64() so the homescreen doesn't get killed. [1] http://androidxref.com/5.1.0_r1/xref/bionic/linker/linker_phdr.cpp#145 [2] http://androidxref.com/5.0.0_r2/xref/bionic/linker/linker_phdr.cpp#145
Hi Michael Wu & Shawn Ku, Please help route this to the appropriate owner. Thanks, Mike
Flags: needinfo?(sku)
Flags: needinfo?(mwu)
Jed, want to take this?
Flags: needinfo?(sku)
Flags: needinfo?(mwu)
Flags: needinfo?(jld)
Assignee: nobody → jld
Flags: needinfo?(jld)
Hardware: x86 → All
Component: GonkIntegration → Security: Process Sandboxing
Product: Firefox OS → Core
Whiteboard: [CR 810045]
Whiteboard: [CR 810045] → [caf priority: p2][CR 810045]
Attached patch Patch (deleted) — — Splinter Review
Attachment #8580203 - Flags: review?(gdestuynder)
Attachment #8580203 - Flags: review?(gdestuynder) → review+
(goto v2.2+, as we do really need this)
blocking-b2g: 2.2? → 2.2+
Comment on attachment 8580203 [details] [diff] [review] Patch NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Bug caused by (feature/regressing bug #): User impact if declined: Testing completed: Risk to taking this patch (and alternatives if risky): String or UUID changes made by this patch:
Attachment #8580203 - Flags: approval-mozilla-b2g37?
waiting for m-c landing before approving on branch.
https://hg.mozilla.org/mozilla-central/rev/5aaf90d7a1e3
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Attachment #8580203 - Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: