Bug 217389

Summary: [Cocoa] The WebContent development XPC service is not allowed to connect to the XPC endpoint in the Networking process
Product: WebKit Reporter: Per Arne Vollan <pvollan>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, ggaren, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
ews-feeder: commit-queue-
Patch
none
Patch
bfulgham: review+
Patch
none
Patch none

Description Per Arne Vollan 2020-10-06 09:01:18 PDT
The WebContent development XPC service is not allowed to connect to the XPC endpoint in the Networking process, because this requires a private entitlement. The WebContent development XPC service is not allowed to have private entitlements. The WebContent process needs to connect to the XPC endpoint in the Networking process in order to receive the Launch Services database XPC object.
Comment 1 Per Arne Vollan 2020-10-06 09:17:31 PDT
Created attachment 410646 [details]
Patch
Comment 2 Per Arne Vollan 2020-10-06 10:09:08 PDT
Created attachment 410652 [details]
Patch
Comment 3 Per Arne Vollan 2020-10-06 10:23:47 PDT
<rdar://problem/68585497>
Comment 4 Per Arne Vollan 2020-10-06 11:36:29 PDT
Created attachment 410670 [details]
Patch
Comment 5 Brent Fulgham 2020-10-06 11:48:43 PDT
Comment on attachment 410670 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=410670&action=review

r=me

> Source/WebKit/Shared/Cocoa/XPCEndpoint.mm:33
> +#import <wtf/text/WTFString.h>

Should most of these things be inside PLATFORM(MAC), since they only seem to be used in that build case?
Comment 6 Per Arne Vollan 2020-10-06 12:34:56 PDT
(In reply to Brent Fulgham from comment #5)
> Comment on attachment 410670 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=410670&action=review
> 
> r=me
> 
> > Source/WebKit/Shared/Cocoa/XPCEndpoint.mm:33
> > +#import <wtf/text/WTFString.h>
> 
> Should most of these things be inside PLATFORM(MAC), since they only seem to
> be used in that build case?

Will fix. Thanks for reviewing!
Comment 7 Per Arne Vollan 2020-10-06 13:01:04 PDT
Created attachment 410682 [details]
Patch
Comment 8 Per Arne Vollan 2020-10-06 13:03:34 PDT
Created attachment 410684 [details]
Patch
Comment 9 Geoffrey Garen 2020-10-06 13:38:00 PDT
Seems like this behavior was always broken in WebContent.Development? If so, and we didn't notice, that's pretty serious. Can we add a regression test for this?
Comment 10 EWS 2020-10-06 13:53:40 PDT
Committed r268072: <https://trac.webkit.org/changeset/268072>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 410684 [details].
Comment 11 Per Arne Vollan 2020-10-06 13:58:51 PDT
(In reply to Geoffrey Garen from comment #9)
> Seems like this behavior was always broken in WebContent.Development? If so,
> and we didn't notice, that's pretty serious. Can we add a regression test
> for this?

Yes, this has been broken for WebContent.Development all along. I can look into creating a test for this.

Thanks for reviewing!
Comment 12 Per Arne Vollan 2020-10-06 14:12:32 PDT
(In reply to Per Arne Vollan from comment #11)
> (In reply to Geoffrey Garen from comment #9)
> > Seems like this behavior was always broken in WebContent.Development? If so,
> > and we didn't notice, that's pretty serious. Can we add a regression test
> > for this?
> 
> Yes, this has been broken for WebContent.Development all along. I can look
> into creating a test for this.
> 
> Thanks for reviewing!

More specifically, it has not been an issue with WebContent in Debug mode, but when loading the system installed version of WebContent.Development.