Summary: | [iOS] Fix mach lookup sandbox violations in the Mail app | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Per Arne Vollan <pvollan> | ||||||||
Component: | WebKit Misc. | Assignee: | Per Arne Vollan <pvollan> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ap, bfulgham, commit-queue, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Per Arne Vollan
2019-10-31 13:59:08 PDT
Created attachment 382499 [details]
Patch
Created attachment 382501 [details]
Patch
Comment on attachment 382501 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382501&action=review > Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:624 > +(allow mach-lookup > (global-name "com.apple.logd.events")) > > (allow mach-lookup > + (global-name "com.apple.distributed_notifications@1v3")) > + > +(allow mach-lookup > + (global-name "com.apple.aggregated")) > + > +(allow mach-lookup > (global-name "com.apple.cfprefsd.daemon")) Should these all be combined into one "allow mach-lookup" expression? Created attachment 382518 [details]
Patch
(In reply to Alexey Proskuryakov from comment #4) > Comment on attachment 382501 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=382501&action=review > > > Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:624 > > +(allow mach-lookup > > (global-name "com.apple.logd.events")) > > > > (allow mach-lookup > > + (global-name "com.apple.distributed_notifications@1v3")) > > + > > +(allow mach-lookup > > + (global-name "com.apple.aggregated")) > > + > > +(allow mach-lookup > > (global-name "com.apple.cfprefsd.daemon")) > > Should these all be combined into one "allow mach-lookup" expression? Done. Thanks for reviewing! Comment on attachment 382518 [details] Patch Clearing flags on attachment: 382518 Committed r251897: <https://trac.webkit.org/changeset/251897> |