Bug 213756 - [iOS] Temporarily open up sandbox for mapping of Launch Services database
Summary: [iOS] Temporarily open up sandbox for mapping of Launch Services database
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Per Arne Vollan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-06-29 15:13 PDT by Per Arne Vollan
Modified: 2020-06-29 15:36 PDT (History)
2 users (show)

See Also:

Attachments
Patch (1.43 KB, patch)
2020-06-29 15:19 PDT, Per Arne Vollan 		bfulgham: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Per Arne Vollan 2020-06-29 15:13:28 PDT 
Temporarily open up sandbox for mapping of Launch Services database in the WebContent process on iOS. This is just a short term fix, and access to the service will be closed again soon.
Comment 1 Per Arne Vollan 2020-06-29 15:16:36 PDT 
rdar://problem/64707000
Comment 2 Per Arne Vollan 2020-06-29 15:19:36 PDT 
Created attachment 403123 [details]
Patch
Comment 3 Brent Fulgham 2020-06-29 15:22:02 PDT 
Comment on attachment 403123 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=403123&action=review

r=me

> Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:591
> +(allow mach-lookup (with telemetry-backtrace)

I would remove the telemetry-backtrace, since this will be getting hit under normal circumstances.
Comment 4 Per Arne Vollan 2020-06-29 15:32:04 PDT 
(In reply to Brent Fulgham from comment #3)
> Comment on attachment 403123 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=403123&action=review
> 
> r=me
> 
> > Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:591
> > +(allow mach-lookup (with telemetry-backtrace)
> 
> I would remove the telemetry-backtrace, since this will be getting hit under
> normal circumstances.

Done.

Thanks for reviewing!