WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
236342
Apply default sandbox to webpushd on Mac
https://bugs.webkit.org/show_bug.cgi?id=236342
Summary
Apply default sandbox to webpushd on Mac
Ben Nham
Reported
2022-02-08 16:48:08 PST
Apply default sandbox to webpushd on Mac
Attachments
Patch
(21.25 KB, patch)
2022-02-08 16:56 PST
,
Ben Nham
no flags
Details
Formatted Diff
Diff
Patch
(20.72 KB, patch)
2022-02-08 17:19 PST
,
Ben Nham
no flags
Details
Formatted Diff
Diff
Patch for landing
(20.54 KB, patch)
2022-02-10 10:10 PST
,
Ben Nham
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Ben Nham
Comment 1
2022-02-08 16:56:52 PST
Created
attachment 451326
[details]
Patch
Ben Nham
Comment 2
2022-02-08 17:19:35 PST
Created
attachment 451328
[details]
Patch
Radar WebKit Bug Importer
Comment 3
2022-02-08 20:29:26 PST
<
rdar://problem/88668389
>
Per Arne Vollan
Comment 4
2022-02-10 09:41:32 PST
Comment on
attachment 451328
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=451328&action=review
R=me.
> Source/WebKit/webpushd/WebPushDaemonMain.mm:95 > + applySandbox();
It might be good to apply the sandbox first thing in WebPushDaemonMain to avoid establishing connections to services which are blocked by the sandbox.
Ben Nham
Comment 5
2022-02-10 10:10:23 PST
Created
attachment 451562
[details]
Patch for landing
EWS
Comment 6
2022-02-10 12:14:37 PST
Committed
r289566
(
247086@main
): <
https://commits.webkit.org/247086@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 451562
[details]
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug