RESOLVED FIXED219638
Remove access to com.apple.audio.toolbox.reporting.service 
https://bugs.webkit.org/show_bug.cgi?id=219638
Summary Remove access to com.apple.audio.toolbox.reporting.service
Per Arne Vollan
Reported 2020-12-08 04:03:55 PST
Remove access to com.apple.audio.toolbox.reporting.service for WebKit processes, since this is a logging service which is not needed for correctness.
Attachments
Patch (4.11 KB, patch)
2020-12-08 04:07 PST, Per Arne Vollan 		darin: review+
DetailsFormatted DiffDiff
Patch (5.15 KB, patch)
2020-12-09 00:27 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2020-12-08 04:04:18 PST
<rdar://problem/71638615>
Per Arne Vollan
Comment 2 2020-12-08 04:07:33 PST
Created attachment 415628 [details] Patch
Darin Adler
Comment 3 2020-12-08 11:45:33 PST
Comment on attachment 415628 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415628&action=review > Source/WebKit/ChangeLog:9 > + Remove access to com.apple.audio.toolbox.reporting.service for WebKit processes, since this is a logging service which is not needed for correctness. In cases like this, do we need to include a line in the sandbox profile that says "prohibit this but don't log it" to save the resources that will otherwise be used to report the sandbox exception?
Per Arne Vollan
Comment 4 2020-12-08 13:08:00 PST
(In reply to Darin Adler from comment #3) > Comment on attachment 415628 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=415628&action=review > > > Source/WebKit/ChangeLog:9 > > + Remove access to com.apple.audio.toolbox.reporting.service for WebKit processes, since this is a logging service which is not needed for correctness. > > In cases like this, do we need to include a line in the sandbox profile that > says "prohibit this but don't log it" to save the resources that will > otherwise be used to report the sandbox exception? That is a good point. I can add that. Thanks for reviewing!
Per Arne Vollan
Comment 5 2020-12-09 00:27:29 PST
Created attachment 415726 [details] Patch
EWS
Comment 6 2020-12-09 00:59:11 PST
Committed r270575: <https://trac.webkit.org/changeset/270575> All reviewed patches have been landed. Closing bug and clearing flags on attachment 415726 [details].
Note You need to log in before you can comment on or make changes to this bug.