Bug 221046

Summary:

[iOS] Fix sandbox violation file-read-data /dev/dtracehelper in the Networking process

Product:

WebKit

Reporter:

Per Arne Vollan <pvollan>

Component:

WebKit Misc.

Assignee:

Per Arne Vollan <pvollan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Per Arne Vollan

Reported 2021-01-27 09:12:37 PST

Reading this file should be allowed in the Networking process on internal installs on iOS.

Attachments
Patch (1.48 KB, patch) 2021-01-27 09:14 PST, Per Arne Vollan	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Per Arne Vollan

Comment 1 2021-01-27 09:13:02 PST

<rdar://problem/73625718>

Per Arne Vollan

Comment 2 2021-01-27 09:14:33 PST

Created attachment 418552 [details] Patch

Brent Fulgham

Comment 3 2021-01-27 09:50:02 PST

Comment on attachment 418552 [details] Patch I think this change looks good, but I think you should take this further and (1) add the apple-internal check in the other places it is used, and (2) add the log silencing check everywhere. You can do that as a follow-up patch is you prefer.

Per Arne Vollan

Comment 4 2021-01-27 10:04:48 PST

(In reply to Brent Fulgham from comment #3) > Comment on attachment 418552 [details] > Patch > > I think this change looks good, but I think you should take this further and > (1) add the apple-internal check in the other places it is used, and (2) add > the log silencing check everywhere. You can do that as a follow-up patch is > you prefer. I will post a follow up patch. Thanks for reviewing!

EWS

Comment 5 2021-01-27 10:07:42 PST

Committed r271964: <https://trac.webkit.org/changeset/271964> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418552 [details].

Note You need to log in before you can comment on or make changes to this bug.