Attachment 342206 Details for Bug 182605 – Patch for landing

[patch] Patch for landing

bug-182605-20180607142225.patch (text/plain), 21.06 KB, created by Tadeu Zagallo on 2018-06-07 14:22:25 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Tadeu Zagallo

Created: 2018-06-07 14:22:25 PDT

Size: 21.06 KB

patch

obsolete

>Subversion Revision: 232546
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 6614868fe35d0a4581dde35291c8d7a4934c8044..b76048568c04b0dea65c14cd4005795c0fa50323 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,18 @@
>+2018-06-07  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Don't try to allocate JIT memory if we don't have the JIT entitlement
>+        https://bugs.webkit.org/show_bug.cgi?id=182605
>+        <rdar://problem/38271229>
>+
>+        Reviewed by Mark Lam.
>+
>+        Check that the current process has the correct entitlements before
>+        trying to allocate JIT memory to silence warnings.
>+
>+        * jit/ExecutableAllocator.cpp:
>+        (JSC::allowJIT): Helper that checks entitlements on iOS and returns true in other platforms
>+        (JSC::FixedVMPoolExecutableAllocator::FixedVMPoolExecutableAllocator): check allowJIT before trying to allocate
>+
> 2018-06-05  Brent Fulgham  <bfulgham@apple.com>
> 
>         Adjust compile and runtime flags to match shippable state of features
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index 05db42447c58fa0bf37e7a8e514eaff731767a85..db3c2ad39476ddeaa8dc457e76b4f35a9a415dd8 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,20 @@
>+2018-06-07  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Don't try to allocate JIT memory if we don't have the JIT entitlement
>+        https://bugs.webkit.org/show_bug.cgi?id=182605
>+        <rdar://problem/38271229>
>+
>+        Reviewed by Mark Lam.
>+
>+        Check that the current process has the correct entitlements before
>+        trying to allocate JIT memory to silence warnings.
>+
>+        * WTF.xcodeproj/project.pbxproj:
>+        * wtf/PlatformMac.cmake:
>+        * wtf/cocoa/Entitlements.cpp:
>+        (WTF::processHasEntitlement): Moved here from Source/WebKit/Shared/mac/SandboxUtilities.mm.
>+        * wtf/cocoa/Entitlements.h: Added.
>+
> 2018-06-05  Darin Adler  <darin@apple.com>
> 
>         [Cocoa] Retire DispatchPtr, and add more move semantics and simpler #ifs to other smart pointers
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index de5206018594b200dcbceadbd79b46ff0d89ff6e..a79e939fdb8076b3de0d295287aaaf81e0becbf7 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,17 @@
>+2018-06-07  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Don't try to allocate JIT memory if we don't have the JIT entitlement
>+        https://bugs.webkit.org/show_bug.cgi?id=182605
>+        <rdar://problem/38271229>
>+
>+        Reviewed by Mark Lam.
>+
>+        Check that the current process has the correct entitlements before
>+        trying to allocate JIT memory to silence warnings.
>+
>+        * Shared/mac/SandboxUtilities.mm:
>+        (WebKit::processHasEntitlement): Move the implementation down to WTF to share with JavaScriptCore
>+
> 2018-06-06  Jeremy Jones  <jeremyj@apple.com>
> 
>         attenuationFactor should be in range [0,1]
>diff --git a/Source/JavaScriptCore/jit/ExecutableAllocator.cpp b/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>index 3e7e7e61fc2cdf6d52a87cf0f96e42f6675b2e0f..4608ae62e32f5a6bd7caa1dd7e77cee4aa23847a 100644
>--- a/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>+++ b/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>@@ -38,6 +38,10 @@
> #include <sys/mman.h>
> #endif
> 
>+#if PLATFORM(IOS)
>+#include <wtf/cocoa/Entitlements.h>
>+#endif
>+
> #include "LinkBuffer.h"
> #include "MacroAssembler.h"
> 
>@@ -110,12 +114,24 @@ JS_EXPORT_PRIVATE JITWriteSeparateHeapsFunction jitWriteSeparateHeapsFunction;
> static uintptr_t startOfFixedWritableMemoryPool;
> #endif
> 
>+static bool allowJIT()
>+{
>+#if PLATFORM(IOS) && (CPU(ARM64) || CPU(ARM))
>+    return processHasEntitlement("dynamic-codesigning");
>+#else
>+    return true;
>+#endif
>+}
>+
> class FixedVMPoolExecutableAllocator : public MetaAllocator {
>     WTF_MAKE_FAST_ALLOCATED;
> public:
>     FixedVMPoolExecutableAllocator()
>         : MetaAllocator(jitAllocationGranule) // round up all allocations to 32 bytes
>     {
>+        if (!allowJIT())
>+            return;
>+
>         size_t reservationSize;
>         if (Options::jitMemoryReservationSize())
>             reservationSize = Options::jitMemoryReservationSize();
>diff --git a/Source/WTF/WTF.xcodeproj/project.pbxproj b/Source/WTF/WTF.xcodeproj/project.pbxproj
>index 862c7ccbc2f08e0c6ee8ee20728156eddef7d3ff..647a9105b77cc40fd1123f0c96a16cb9053763ee 100644
>--- a/Source/WTF/WTF.xcodeproj/project.pbxproj
>+++ b/Source/WTF/WTF.xcodeproj/project.pbxproj
>@@ -44,6 +44,7 @@
> 		0FEC3C5E1F368A9700F59B6C /* ReadWriteLock.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FEC3C5C1F368A9700F59B6C /* ReadWriteLock.cpp */; };
> 		0FFF19DC1BB334EB00886D91 /* ParallelHelperPool.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FFF19DA1BB334EB00886D91 /* ParallelHelperPool.cpp */; };
> 		14022F4118F5C3FC007FF0EB /* libbmalloc.a in Frameworks */ = {isa = PBXBuildFile; fileRef = 14022F4018F5C3FC007FF0EB /* libbmalloc.a */; };
>+		143DDE9620C8BC37007F76FA /* Entitlements.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 143DDE9520C8BC37007F76FA /* Entitlements.cpp */; };
> 		143F611F1565F0F900DB514A /* RAMSize.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 143F611D1565F0F900DB514A /* RAMSize.cpp */; };
> 		1447AEC618FCE57700B3D7FF /* Foundation.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 1447AEC518FCE57700B3D7FF /* Foundation.framework */; };
> 		1447AEC718FCE58000B3D7FF /* libWTF.a in Frameworks */ = {isa = PBXBuildFile; fileRef = 5D247B6214689B8600E78B76 /* libWTF.a */; };
>@@ -268,6 +269,8 @@
> 		0FFF19DB1BB334EB00886D91 /* ParallelHelperPool.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ParallelHelperPool.h; sourceTree = "<group>"; };
> 		132743924FC54E469F5A8E6E /* StdUnorderedSet.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = StdUnorderedSet.h; sourceTree = "<group>"; };
> 		14022F4018F5C3FC007FF0EB /* libbmalloc.a */ = {isa = PBXFileReference; lastKnownFileType = archive.ar; path = libbmalloc.a; sourceTree = BUILT_PRODUCTS_DIR; };
>+		143DDE9520C8BC37007F76FA /* Entitlements.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = Entitlements.cpp; sourceTree = "<group>"; };
>+		143DDE9720C8BE99007F76FA /* Entitlements.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Entitlements.h; sourceTree = "<group>"; };
> 		143F611D1565F0F900DB514A /* RAMSize.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = RAMSize.cpp; sourceTree = "<group>"; };
> 		143F611E1565F0F900DB514A /* RAMSize.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RAMSize.h; sourceTree = "<group>"; };
> 		1447AEBE18FCE56900B3D7FF /* libmbmalloc.dylib */ = {isa = PBXFileReference; explicitFileType = "compiled.mach-o.dylib"; includeInIndex = 0; path = libmbmalloc.dylib; sourceTree = BUILT_PRODUCTS_DIR; };
>@@ -1315,6 +1318,8 @@
> 			isa = PBXGroup;
> 			children = (
> 				E38C41241EB4E04C0042957D /* CPUTimeCocoa.mm */,
>+				143DDE9520C8BC37007F76FA /* Entitlements.cpp */,
>+				143DDE9720C8BE99007F76FA /* Entitlements.h */,
> 				7A6EBA3320746C34004F9C44 /* MachSendRight.cpp */,
> 				ADF2CE651E39F106006889DB /* MemoryFootprintCocoa.cpp */,
> 				AD89B6B91E64150F0090707F /* MemoryPressureHandlerCocoa.mm */,
>@@ -1465,6 +1470,7 @@
> 				A8A473AE151A825B004123FF /* diy-fp.cc in Sources */,
> 				A8A473B0151A825B004123FF /* double-conversion.cc in Sources */,
> 				A8A473BA151A825B004123FF /* dtoa.cpp in Sources */,
>+				143DDE9620C8BC37007F76FA /* Entitlements.cpp in Sources */,
> 				A8A473B3151A825B004123FF /* fast-dtoa.cc in Sources */,
> 				0F7C5FB61D885CF20044F5E2 /* FastBitVector.cpp in Sources */,
> 				A8A473C3151A825B004123FF /* FastMalloc.cpp in Sources */,
>diff --git a/Source/WTF/wtf/PlatformMac.cmake b/Source/WTF/wtf/PlatformMac.cmake
>index 74b8dd906ad8aec9bfce03ce825bfee02c60e495..a26e0503e5f35e994f2c39b3991f3d8ce644b356 100644
>--- a/Source/WTF/wtf/PlatformMac.cmake
>+++ b/Source/WTF/wtf/PlatformMac.cmake
>@@ -10,6 +10,7 @@ list(APPEND WTF_LIBRARIES
> list(APPEND WTF_PUBLIC_HEADERS
>     cf/TypeCastsCF.h
> 
>+    cocoa/Entitlements.h
>     cocoa/MachSendRight.h
>     cocoa/SoftLinking.h
> 
>@@ -43,6 +44,7 @@ list(APPEND WTF_SOURCES
>     text/mac/TextBreakIteratorInternalICUMac.mm
> 
>     cocoa/CPUTimeCocoa.mm
>+    cocoa/Entitlements.cpp
>     cocoa/MachSendRight.cpp
>     cocoa/MemoryFootprintCocoa.cpp
>     cocoa/MemoryPressureHandlerCocoa.mm
>diff --git a/Source/WTF/wtf/cocoa/Entitlements.cpp b/Source/WTF/wtf/cocoa/Entitlements.cpp
>new file mode 100644
>index 0000000000000000000000000000000000000000..ecad951d01daff034bfb03f684c3c95456ba107d
>--- /dev/null
>+++ b/Source/WTF/wtf/cocoa/Entitlements.cpp
>@@ -0,0 +1,51 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#include "config.h"
>+#include "Entitlements.h"
>+
>+#include <wtf/RetainPtr.h>
>+#include <wtf/spi/cocoa/SecuritySPI.h>
>+
>+namespace WTF {
>+
>+bool processHasEntitlement(const char* entitlement)
>+{
>+    auto task = adoptCF(SecTaskCreateFromSelf(kCFAllocatorDefault));
>+    if (!task)
>+        return false;
>+
>+    auto cfEntitlement = adoptCF(CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, entitlement, kCFStringEncodingUTF8, kCFAllocatorNull));
>+    auto value = adoptCF(SecTaskCopyValueForEntitlement(task.get(), cfEntitlement.get(), nullptr));
>+    if (!value)
>+        return false;
>+
>+    if (CFGetTypeID(value.get()) != CFBooleanGetTypeID())
>+        return false;
>+
>+    return CFBooleanGetValue(static_cast<CFBooleanRef>(value.get()));
>+}
>+
>+} // namespace WTF
>diff --git a/Source/WTF/wtf/cocoa/Entitlements.h b/Source/WTF/wtf/cocoa/Entitlements.h
>new file mode 100644
>index 0000000000000000000000000000000000000000..46a9a2468abf8860634e3544cb89417bb541a2f8
>--- /dev/null
>+++ b/Source/WTF/wtf/cocoa/Entitlements.h
>@@ -0,0 +1,34 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#if PLATFORM(COCOA)
>+
>+namespace WTF {
>+
>+WTF_EXPORT bool processHasEntitlement(const char* entitlement);
>+
>+} // namespace WTF
>+
>+#endif
>diff --git a/Source/WTF/wtf/spi/cocoa/SecuritySPI.h b/Source/WTF/wtf/spi/cocoa/SecuritySPI.h
>index 9644a6115de1f291536e0fc8d54a14000990c5f2..96e30a7c7d294f455c03b0de37be0a4d12135e42 100644
>--- a/Source/WTF/wtf/spi/cocoa/SecuritySPI.h
>+++ b/Source/WTF/wtf/spi/cocoa/SecuritySPI.h
>@@ -37,6 +37,8 @@
> 
> #else
> 
>+#include <Security/SecBase.h>
>+
> typedef uint32_t SecSignatureHashAlgorithm;
> enum {
>     kSecSignatureHashAlgorithmUnknown = 0,
>@@ -59,6 +61,7 @@ WTF_EXTERN_C_END
> #endif
> 
> typedef struct __SecTask *SecTaskRef;
>+typedef struct __SecTrust *SecTrustRef;
> 
> WTF_EXTERN_C_BEGIN
> 
>diff --git a/Source/WebKit/Shared/mac/SandboxUtilities.h b/Source/WebKit/Shared/mac/SandboxUtilities.h
>index 4f5fbcdd7eaae47e79daacf1ba8991cbff3c3c40..d60f427d6797873e32222b62b565302c95e636ab 100644
>--- a/Source/WebKit/Shared/mac/SandboxUtilities.h
>+++ b/Source/WebKit/Shared/mac/SandboxUtilities.h
>@@ -38,7 +38,6 @@ bool processHasContainer();
> // Returns an empty string if the process is not in a container.
> String pathForProcessContainer();
> 
>-bool processHasEntitlement(NSString *entitlement);
> bool connectedProcessHasEntitlement(xpc_connection_t, const char *entitlement);
> 
> }
>diff --git a/Source/WebKit/Shared/mac/SandboxUtilities.mm b/Source/WebKit/Shared/mac/SandboxUtilities.mm
>index d386667b9813d41bc89a67a528a76226a1e6774e..afc6b24ae1d7e889eac073f6cd96d0422d6231bb 100644
>--- a/Source/WebKit/Shared/mac/SandboxUtilities.mm
>+++ b/Source/WebKit/Shared/mac/SandboxUtilities.mm
>@@ -29,7 +29,7 @@
> #import <array>
> #import <sys/param.h>
> #import <wtf/OSObjectPtr.h>
>-#import <wtf/spi/cocoa/SecuritySPI.h>
>+#import <wtf/cocoa/Entitlements.h>
> #import <wtf/spi/darwin/SandboxSPI.h>
> #import <wtf/spi/darwin/XPCSPI.h>
> #import <wtf/text/WTFString.h>
>@@ -77,22 +77,6 @@ String pathForProcessContainer()
>     return String::fromUTF8(path.data());
> }
> 
>-bool processHasEntitlement(NSString *entitlement)
>-{
>-    auto task = adoptCF(SecTaskCreateFromSelf(CFAllocatorGetDefault()));
>-    if (!task)
>-        return false;
>-
>-    auto value = adoptCF(SecTaskCopyValueForEntitlement(task.get(), (__bridge CFStringRef)entitlement, nullptr));
>-    if (!value)
>-        return false;
>-
>-    if (CFGetTypeID(value.get()) != CFBooleanGetTypeID())
>-        return false;
>-
>-    return CFBooleanGetValue(static_cast<CFBooleanRef>(value.get()));
>-}
>-
> bool connectedProcessHasEntitlement(xpc_connection_t connection, const char *entitlement)
> {
>     auto value = adoptOSObject(xpc_connection_copy_entitlement_value(connection, entitlement));
>diff --git a/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm b/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>index 90b88900077c245f89f4f7e030b2573d2aa3784a..e0a0f9ad75968d791f26289fb0b91c24a3c037a2 100644
>--- a/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>+++ b/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>@@ -46,7 +46,6 @@
> #import "RemoteLayerTreeTransaction.h"
> #import "RemoteObjectRegistry.h"
> #import "RemoteObjectRegistryMessages.h"
>-#import "SandboxUtilities.h"
> #import "UIDelegate.h"
> #import "UserMediaProcessManager.h"
> #import "VersionChecks.h"
>@@ -151,6 +150,7 @@
> #import <WebCore/WebSQLiteDatabaseTrackerClient.h>
> #import <pal/spi/cg/CoreGraphicsSPI.h>
> #import <pal/spi/cocoa/QuartzCoreSPI.h>
>+#import <wtf/cocoa/Entitlements.h>
> 
> #define RELEASE_LOG_IF_ALLOWED(...) RELEASE_LOG_IF(_page && _page->isAlwaysOnLoggingAllowed(), ViewState, __VA_ARGS__)
> 
>@@ -616,7 +616,7 @@ - (void)_initializeWithConfiguration:(WKWebViewConfiguration *)configuration
> #endif
> 
> #if PLATFORM(IOS) && ENABLE(SERVICE_WORKER)
>-    if (!WebKit::processHasEntitlement(@"com.apple.developer.WebKit.ServiceWorkers"))
>+    if (!WTF::processHasEntitlement("com.apple.developer.WebKit.ServiceWorkers"))
>         pageConfiguration->preferenceValues().set(WebKit::WebPreferencesKey::serviceWorkersEnabledKey(), WebKit::WebPreferencesStore::Value(false));
> #endif
> 
>diff --git a/Source/WebKit/UIProcess/ApplicationStateTracker.mm b/Source/WebKit/UIProcess/ApplicationStateTracker.mm
>index c6523a38ad863f8fbc52e4d182e0e87a0c779836..1cfa080b88ed98f0728e8e71c59fd67d413ea767 100644
>--- a/Source/WebKit/UIProcess/ApplicationStateTracker.mm
>+++ b/Source/WebKit/UIProcess/ApplicationStateTracker.mm
>@@ -32,6 +32,7 @@
> #import "SandboxUtilities.h"
> #import "UIKitSPI.h"
> #import <wtf/ObjcRuntimeExtras.h>
>+#import <wtf/cocoa/Entitlements.h>
> #import <wtf/spi/cocoa/SecuritySPI.h>
> 
> @interface UIWindow (WKDetails)
>@@ -54,7 +55,7 @@ static ApplicationType applicationType(UIWindow *window)
>     if (_UIApplicationIsExtension())
>         return ApplicationType::Extension;
> 
>-    if (processHasEntitlement(@"com.apple.UIKit.vends-view-services") && window._isHostedInAnotherProcess)
>+    if (WTF::processHasEntitlement("com.apple.UIKit.vends-view-services") && window._isHostedInAnotherProcess)
>         return ApplicationType::ViewService;
> 
>     return ApplicationType::Application;
>diff --git a/Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm b/Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm
>index d4557ec013e3ca8c53045748366f1909635ca790..3def2291715a1d4775e6b6769590699493cc61b3 100644
>--- a/Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm
>+++ b/Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm
>@@ -29,7 +29,6 @@
> #if PLATFORM(IOS)
> 
> #import "APIUIClient.h"
>-#import "SandboxUtilities.h"
> #import "TCCSPI.h"
> #import "UIKitSPI.h"
> #import "WKActionSheet.h"
>@@ -45,6 +44,7 @@
> #import <WebCore/WebCoreNSURLExtras.h>
> #import <wtf/SoftLinking.h>
> #import <wtf/WeakObjCPtr.h>
>+#import <wtf/cocoa/Entitlements.h>
> #import <wtf/text/WTFString.h>
> 
> #if HAVE(APP_LINKS)
>@@ -66,7 +66,7 @@ using namespace WebKit;
> #if HAVE(APP_LINKS)
> static bool applicationHasAppLinkEntitlements()
> {
>-    static bool hasEntitlement = processHasEntitlement(@"com.apple.private.canGetAppLinkInfo") && processHasEntitlement(@"com.apple.private.canModifyAppLinkPermissions");
>+    static bool hasEntitlement = WTF::processHasEntitlement("com.apple.private.canGetAppLinkInfo") && WTF::processHasEntitlement("com.apple.private.canModifyAppLinkPermissions");
>     return hasEntitlement;
> }
> 
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 89b532a386d6e602ff1b3ca2fff9e0cc180baad4..e64a4d728fddcb2c94a3815b94d178a4f37d1b0a 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-06-07  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Don't try to allocate JIT memory if we don't have the JIT entitlement
>+        https://bugs.webkit.org/show_bug.cgi?id=182605
>+        <rdar://problem/38271229>
>+
>+        Reviewed by Mark Lam.
>+
>+        Check that the current process has the correct entitlements before
>+        trying to allocate JIT memory to silence warnings.
>+
>+        * TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: iOS requires the Security framework to check entitlements
>+
> 2018-06-05  Chris Dumez  <cdumez@apple.com>
> 
>         Regression(r232082): Websites get loaded inside of Messages App chat transcript
>diff --git a/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj b/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>index 13167a892a4bf946b80ab55a7b56a710e80919fc..590720d4920d461e979555a9d3ef10aa2da8a6fc 100644
>--- a/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>+++ b/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>@@ -47,6 +47,7 @@
> 		0F5651F71FCE4DDC00310FBC /* NoHistoryItemScrollToFragment.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0F5651F61FCE4DDB00310FBC /* NoHistoryItemScrollToFragment.mm */; };
> 		0F5651F91FCE513500310FBC /* scroll-to-anchor.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 0F5651F81FCE50E800310FBC /* scroll-to-anchor.html */; };
> 		115EB3431EE0BA03003C2C0A /* ViewportSizeForViewportUnits.mm in Sources */ = {isa = PBXBuildFile; fileRef = 115EB3421EE0B720003C2C0A /* ViewportSizeForViewportUnits.mm */; };
>+		143DDE9820C9018B007F76FA /* Security.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 574F55D0204D471C002948C6 /* Security.framework */; };
> 		1A02C870125D4CFD00E3F4BD /* find.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 1A02C84B125D4A5E00E3F4BD /* find.html */; };
> 		1A3524AE1D63A4FB0031729B /* Scope.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 1A3524AC1D63A4FB0031729B /* Scope.cpp */; };
> 		1A4F81CF1BDFFD53004E672E /* RemoteObjectRegistryPlugIn.mm in Sources */ = {isa = PBXBuildFile; fileRef = 1A4F81CD1BDFFD53004E672E /* RemoteObjectRegistryPlugIn.mm */; };
>@@ -2020,6 +2021,7 @@
> 			buildActionMask = 2147483647;
> 			files = (
> 				93F56DA71E5F9174003EDE84 /* libicucore.dylib in Frameworks */,
>+				143DDE9820C9018B007F76FA /* Security.framework in Frameworks */,
> 			);
> 			runOnlyForDeploymentPostprocessing = 0;
> 		};

Actions: View | Formatted Diff | Diff

Attachments on bug 182605: 342190 | 342202 | 342206 | 342208 | 342212