Bug 230715

Summary: Remove access to unused services 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: achristensen, bfulgham, gavin.p, mazander, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Per Arne Vollan
Reported 2021-09-23 13:30:48 PDT
Telemetry is showing that some services allowed in the Networking process' sandbox are unused. These should be removed.
Attachments
Patch (2.28 KB, patch)
2021-09-23 13:36 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-09-23 13:31:33 PDT
<rdar://problem/83462947>
Per Arne Vollan
Comment 2 2021-09-23 13:36:30 PDT
Created attachment 439087 [details] Patch
Brent Fulgham
Comment 3 2021-09-23 14:43:20 PDT
Comment on attachment 439087 [details] Patch r=me
Per Arne Vollan
Comment 4 2021-09-23 14:49:06 PDT
Comment on attachment 439087 [details] Patch Thanks for reviewing!
EWS
Comment 5 2021-09-23 14:57:28 PDT
Committed r283012 (242077@main): <https://commits.webkit.org/242077@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 439087 [details].
Simon Fraser (smfr)
Comment 6 2021-09-23 14:59:44 PDT
Comment on attachment 439087 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=439087&action=review > Source/WebKit/ChangeLog:3 > + Remove access to unused services in the Networking access Bug title was right. Changelog was wrong.
Per Arne Vollan
Comment 7 2021-09-23 15:19:37 PDT
(In reply to Simon Fraser (smfr) from comment #6) > Comment on attachment 439087 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=439087&action=review > > > Source/WebKit/ChangeLog:3 > > + Remove access to unused services in the Networking access > > Bug title was right. Changelog was wrong. Ah, thanks, good catch :)
Note You need to log in before you can comment on or make changes to this bug.