Bug 231694

Summary: [macOS][GPUP] Remove access in sandbox to com.apple.audio.AudioComponentRegistrar
Product: WebKit Reporter: Per Arne Vollan <pvollan>
Component: WebKit Misc.Assignee: Per Arne Vollan <pvollan>
Status: REOPENED    
Severity: Normal CC: bfulgham, commit-queue, eric.carlson, ews-watchlist, gavin.p, glenn, jer.noble, mazander, philipj, sergio, sihui_liu, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 232864, 233724    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
none
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
none
Patch
none
Patch
none
Patch none

Per Arne Vollan
Reported 2021-10-13 12:51:58 PDT
After https://trac.webkit.org/changeset/274435/webkit, there is no need to allow access to this service.
Attachments
Patch (2.03 KB, patch)
2021-10-13 12:54 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (19.94 KB, patch)
2021-10-15 09:46 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (20.91 KB, patch)
2021-10-15 09:58 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (13.47 KB, patch)
2021-10-19 13:10 PDT, Per Arne Vollan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (20.01 KB, patch)
2021-10-19 14:16 PDT, Per Arne Vollan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (32.82 KB, patch)
2021-10-21 09:16 PDT, Per Arne Vollan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (14.16 KB, patch)
2021-11-08 09:48 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (14.20 KB, patch)
2021-11-08 09:54 PST, Per Arne Vollan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (19.50 KB, patch)
2021-11-08 09:57 PST, Per Arne Vollan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (19.39 KB, patch)
2021-11-08 10:34 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (19.50 KB, patch)
2021-11-17 11:11 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (21.99 KB, patch)
2022-01-24 14:49 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (21.20 KB, patch)
2022-01-29 13:29 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (9) View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2021-10-13 12:54:38 PDT
Created attachment 441121 [details] Patch
Per Arne Vollan
Comment 2 2021-10-13 12:55:30 PDT
<rdar://75225923>
Brent Fulgham
Comment 3 2021-10-13 14:39:22 PDT
Comment on attachment 441121 [details] Patch r=me
Per Arne Vollan
Comment 4 2021-10-15 09:46:22 PDT
Created attachment 441387 [details] Patch
Per Arne Vollan
Comment 5 2021-10-15 09:47:19 PDT
(In reply to Brent Fulgham from comment #3) > Comment on attachment 441121 [details] > Patch > > r=me I have uploaded a revised patch for review. Thanks for reviewing!
Per Arne Vollan
Comment 6 2021-10-15 09:58:21 PDT
Created attachment 441391 [details] Patch
Brent Fulgham
Comment 7 2021-10-15 15:21:50 PDT
Comment on attachment 441391 [details] Patch r=me
Brent Fulgham
Comment 8 2021-10-15 15:22:27 PDT
Can you clean up the error in the header file noted by the style check script?
Per Arne Vollan
Comment 9 2021-10-19 13:10:26 PDT
Created attachment 441779 [details] Patch
Per Arne Vollan
Comment 10 2021-10-19 13:18:33 PDT
(In reply to Brent Fulgham from comment #8) > Can you clean up the error in the header file noted by the style check > script? In the current form of the patch, I believe this include is needed, since the implementation of the template should be in the header file. I will look closer into fixing this style error. Thanks for reviewing!
Per Arne Vollan
Comment 11 2021-10-19 14:16:20 PDT
Created attachment 441797 [details] Patch
Per Arne Vollan
Comment 12 2021-10-21 09:16:54 PDT
Created attachment 442039 [details] Patch
Per Arne Vollan
Comment 13 2021-11-08 09:48:54 PST
Created attachment 443559 [details] Patch
Per Arne Vollan
Comment 14 2021-11-08 09:54:28 PST
Created attachment 443560 [details] Patch
Per Arne Vollan
Comment 15 2021-11-08 09:57:10 PST
Created attachment 443563 [details] Patch
Per Arne Vollan
Comment 16 2021-11-08 10:34:44 PST
Created attachment 443569 [details] Patch
EWS
Comment 17 2021-11-08 14:28:31 PST
Committed r285432 (243988@main): <https://commits.webkit.org/243988@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 443569 [details].
WebKit Commit Bot
Comment 18 2021-11-08 20:21:12 PST
Re-opened since this is blocked by bug 232864
Per Arne Vollan
Comment 19 2021-11-17 11:11:10 PST
Created attachment 444538 [details] Patch
EWS
Comment 20 2021-11-18 15:54:27 PST
Committed r286037 (244425@main): <https://commits.webkit.org/244425@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444538 [details].
WebKit Commit Bot
Comment 21 2021-12-01 13:59:59 PST
Re-opened since this is blocked by bug 233724
Per Arne Vollan
Comment 22 2021-12-01 14:11:45 PST
Note to self: this should be tested on test.webrtc.org.
Per Arne Vollan
Comment 23 2022-01-24 14:49:10 PST
Created attachment 449861 [details] Patch
Per Arne Vollan
Comment 24 2022-01-29 13:29:32 PST
Created attachment 450334 [details] Patch
Note You need to log in before you can comment on or make changes to this bug.