Loading...
tests/enablement_tests.m libmalloc-792.60.6 libmalloc-715.120.13
--- libmalloc/libmalloc-792.60.6/tests/enablement_tests.m
+++ libmalloc/libmalloc-715.120.13/tests/enablement_tests.m
@@ -9,15 +9,13 @@
 #include <darwintest_utils.h>
 #include <../src/internal.h>
 
-#if CONFIG_XZONE_MALLOC && (MALLOC_TARGET_IOS_ONLY || TARGET_OS_OSX || TARGET_OS_VISION || TARGET_OS_WATCH)
+#if CONFIG_XZONE_MALLOC && (MALLOC_TARGET_IOS_ONLY || TARGET_OS_OSX || TARGET_OS_VISION)
 
 #include <Foundation/Foundation.h>
 #include <Foundation/NSJSONSerialization.h>
 
 #define PID_BUFFER_SIZE 256
 #define NUM_PIDS_BUFFER_SIZE (512 * sizeof (pid_t))
-
-#define ALLOCATION_FRONT_EXTRA 1
 
 static char *print_buffer = NULL;
 static size_t print_buffer_capacity = 0;
@@ -264,45 +262,29 @@
 	return devicename;
 }
 
-static bool
-get_device_should_defer_large(void)
-{
-	bool should_defer_large = false;
-
+static void
+enablement_configuration_process_checks(
+	NSArray *json_array,
+	enablement_configuration *configuration
+	)
+{
 #if MALLOC_TARGET_IOS_ONLY
-	char device_name[16] = { 0 };
-	get_device_name(device_name, sizeof(device_name) - 1);
-
-	T_LOG("Device name: %s", device_name);
-
-	if (!strcmp(device_name, "J420") || !strcmp(device_name, "J421")) {
-		return false;
+	// Array of all the non-AMP hardware
+	char non_amp_hardware[2][16] = {"J171", "J172"};
+	char devicename[16] = { 0 };
+	size_t len = sizeof(devicename) - 1;
+	get_device_name(devicename, len);
+
+	// The secure allocator should not be enabled for any process on non-AMP
+	// hardware
+	for (int i = 0; i < countof(non_amp_hardware); ++i) {
+		if (!strcmp(devicename, non_amp_hardware[i])) {
+			T_EXPECT_EQ(json_array.count, 0ul, "No zones should be present");
+			return;
+		}
 	}
-
-	// If an iOS device has >=6GB of memory, the enablement configuration
-	// should have defer_large set to "true".  Note that here we check for >=
-	// 5GB because in practice various carve-outs reduce the actual size of
-	// various 6GB devices to a bit below that quantity.  There are no devices
-	// with >= 5GB of memory on iOS that aren't actually 6GB devices in the
-	// sense we mean here.
-	uint64_t memsize = platform_hw_memsize();
-	const uint64_t defer_large_ios_bytes_memsize = 5 * 1073741824ULL;
-	T_LOG("Device memsize: %"PRIu64", defer_large_ios_bytes_memsize: %"PRIu64"",
-			memsize, defer_large_ios_bytes_memsize);
-	if (memsize >= defer_large_ios_bytes_memsize) {
-		should_defer_large = true;
-	}
-#elif TARGET_OS_OSX
-	should_defer_large = true;
-#endif
-
-	return should_defer_large;
-}
-
-static void
-enablement_configuration_process_checks(NSArray *json_array,
-		enablement_configuration *configuration)
-{
+#endif
+
 	// The secure allocator should not be enabled for any process on intel
 	// machines
 #if TARGET_OS_OSX && !TARGET_CPU_ARM64
@@ -356,11 +338,11 @@
 }
 
 static pid_t
-spawn_process(char *new_argv[], char *new_envp[])
+spawn_process(char *new_argv[])
 {
 	pid_t child_pid = 0;
 	errno_t ret = posix_spawn(&child_pid, new_argv[0], NULL, NULL,
-			new_argv, new_envp);
+			new_argv, NULL);
 	T_ASSERT_POSIX_ZERO(ret, "posix_spawn(%s)", new_argv[0]);
 	T_ASSERT_NE(child_pid, 0, "posix_spawn(%s)", new_argv[0]);
 
@@ -373,47 +355,7 @@
 }
 
 static void
-security_critical_configuration_checks_with_space_efficiency(
-		const char *process, bool space_efficient)
-{
-	struct enablement_configuration configuration = (enablement_configuration) {
-		.should_have_xzones = true,
-		.guards_enabled = true,
-#if TARGET_OS_VISION
-		.batch_size = 0,
-		.ptr_bucket_count = 4,
-#elif TARGET_OS_OSX
-		.batch_size = space_efficient ? 0 : 10,
-		.ptr_bucket_count = 4,
-#elif TARGET_OS_WATCH
-		.batch_size = 0,
-		.ptr_bucket_count = 2,
-#else
-		.batch_size = 0,
-		.ptr_bucket_count = 3,
-#endif
-		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.thread_cache_enabled = !space_efficient,
-#if TARGET_OS_OSX
-		.segment_group_count = (space_efficient ? 1 : get_ncpuclusters()) *
-				(XZM_SEGMENT_GROUP_IDS_COUNT + ALLOCATION_FRONT_EXTRA),
-		.defer_tiny = !space_efficient,
-		.defer_small = !space_efficient,
-#else
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
-		.defer_tiny = false,
-		.defer_small = false,
-#endif // TARGET_OS_OSX
-		.defer_large = (!space_efficient && get_device_should_defer_large()),
-	 };
-
-	enablement_configuration_process_checks(get_process_json(process, false),
-			&configuration);
-}
-
-static void
-security_critical_configuration_checks(const char *process)
+security_critical_configuration_checks(char *process)
 {
 	struct enablement_configuration configuration = (enablement_configuration) {
 		.should_have_xzones = true,
@@ -422,29 +364,27 @@
 		.batch_size = 0,
 #if TARGET_OS_OSX || TARGET_OS_VISION
 		.ptr_bucket_count = 4,
-#elif TARGET_OS_WATCH
-		.ptr_bucket_count = 2,
 #else
 		.ptr_bucket_count = 3,
 #endif
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = 1 * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = false,
 		.defer_small = false,
 		.defer_large = false,
 	 };
 
-	enablement_configuration_process_checks(get_process_json(process, false),
-			&configuration);
-}
-
-void terminate_process(pid_t pid)
-{
-	T_EXPECT_POSIX_SUCCESS(kill(pid, SIGKILL), "terminated process");
-}
-
-T_GLOBAL_META(T_META_TAG_NO_ALLOCATOR_OVERRIDE);
+	enablement_configuration_process_checks(
+		get_process_json(process, false),
+		&configuration
+		);
+}
+
+void terminate_process(pid_t pid) {
+	char terminate_process_buffer[PID_BUFFER_SIZE] = {};
+	snprintf(terminate_process_buffer, PID_BUFFER_SIZE, "kill -9 %d", pid);
+	T_ASSERT_POSIX_ZERO(system(terminate_process_buffer), "terminated process");
+}
 
 T_DECL(xzone_enabled_launchd,
 		"Verify enablement configuration for security critical processes"
@@ -471,8 +411,6 @@
 {
 	security_critical_configuration_checks("notifyd");
 }
-
-#if !TARGET_OS_WATCH
 
 // This test needs to be run as the local (non-root) user on macOS in order to
 // successfully launch Safari
@@ -488,7 +426,7 @@
 	// Launch the Safari process on macOS
 	char *launch_safari_args[] = {"/usr/bin/open", "-a", "Safari",
 	"http://apple.com", NULL};
-	pid_t safari_pid = spawn_process(launch_safari_args, NULL);
+	pid_t safari_pid = spawn_process(launch_safari_args);
 	security_critical_configuration_checks("Safari");
 #else
 #if MALLOC_TARGET_IOS_ONLY
@@ -510,11 +448,6 @@
 	terminate_process(safari_pid);
 #endif
 }
-
-#endif // !TARGET_OS_WATCH
-
-// TODO: port this test to watchOS
-#if !TARGET_OS_WATCH
 
 T_DECL(xzone_enabled_driverkit,
 		"Verify enablement configuration for Driverkit processes",
@@ -526,7 +459,7 @@
 	// processes running by default. Thus, run a simple test to start the
 	// com.apple.AppleUserHIDDriver process for analysis.
 	char *spawn_driverkit_proc_args[] = {"/usr/local/bin/hidUserDeviceTest", "hidUserDeviceTest", "-k", NULL};
-	pid_t driver_test_pid = spawn_process(spawn_driverkit_proc_args, NULL);
+	pid_t driver_test_pid = spawn_process(spawn_driverkit_proc_args);
 
 	// The above action would have started the DriverKit process with the
 	// label: com.apple.AppleUserHIDDriver
@@ -537,14 +470,11 @@
 		.batch_size = 0,
 #if TARGET_OS_OSX || TARGET_OS_VISION
 		.ptr_bucket_count = 4,
-#elif TARGET_OS_WATCH
-		.ptr_bucket_count = 2,
 #else
 		.ptr_bucket_count = 3,
 #endif
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = 1 * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = false,
 		.defer_small = false,
 		.defer_large = false,
@@ -558,8 +488,6 @@
 	terminate_process(driver_test_pid);
 }
 
-#endif // !TARGET_OS_WATCH
-
 T_DECL(xzone_enabled_general_process_test_runner,
 		"Verify enablement configuration for general processes (the test" "process itself)",
 		T_META_TAG_VM_NOT_ELIGIBLE,
@@ -567,37 +495,24 @@
 		T_META_ASROOT(true))
 {
 	struct enablement_configuration configuration = (enablement_configuration) {
+#if TARGET_OS_OSX
+		.should_have_xzones = false,
+#else
 		.should_have_xzones = true,
+#endif
 		.guards_enabled = false,
-#if TARGET_OS_OSX
-		.thread_cache_enabled = true,
-		.batch_size = 10,
-#else
 		.thread_cache_enabled = false,
 		.batch_size = 0,
-#endif
-#if TARGET_OS_OSX || TARGET_OS_VISION
+#if TARGET_OS_VISION
 		.ptr_bucket_count = 4,
-#elif TARGET_OS_WATCH
-		.ptr_bucket_count = 1,
 #else
 		.ptr_bucket_count = 2,
 #endif
-#if TARGET_OS_OSX
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = get_ncpuclusters() *
-				(XZM_SEGMENT_GROUP_IDS_COUNT + ALLOCATION_FRONT_EXTRA),
-		.defer_tiny = true,
-		.defer_small = true,
-		.defer_large = true,
-#else
-		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = 1 * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = false,
 		.defer_small = false,
 		.defer_large = false,
-#endif
 	 };
 
 	enablement_configuration_process_checks(
@@ -607,7 +522,7 @@
 }
 
 T_DECL(xzone_enabled_general_daemon,
-		"Verify enablement configuration for general daemon (watchdogd)",
+		"Verify enablement configuration for general processes (a daemon," "watchdogd)",
 		T_META_TAG_VM_NOT_ELIGIBLE,
 		T_META_ASROOT(true))
 {
@@ -618,14 +533,11 @@
 		.batch_size = 0,
 #if TARGET_OS_VISION || TARGET_OS_OSX
 		.ptr_bucket_count = 4,
-#elif TARGET_OS_WATCH
-		.ptr_bucket_count = 1,
 #else
 		.ptr_bucket_count = 2,
 #endif
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = 1 * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = false,
 		.defer_small = false,
 		.defer_large = false,
@@ -650,8 +562,7 @@
 		.batch_size = 10,
 		.ptr_bucket_count = 4,
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-		.segment_group_count = get_ncpuclusters() *
-				(XZM_SEGMENT_GROUP_IDS_COUNT + ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = get_ncpuclusters() * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = true,
 		.defer_small = true,
 		.defer_large = true,
@@ -659,7 +570,7 @@
 
 	// Launch the Messages app on macOS
 	char *launch_notes_args[] = {"/System/Applications/Messages.app/Contents/MacOS/Messages", NULL};
-	pid_t pid = spawn_process(launch_notes_args, NULL);
+	pid_t pid = spawn_process(launch_notes_args);
 
 	// On macOS, we expect this to use xzone malloc
 	enablement_configuration_process_checks(
@@ -671,48 +582,48 @@
 }
 #endif // TARGET_OS_OSX
 
-#if !TARGET_OS_WATCH
-
 T_DECL(xzone_enabled_general_app,
 		"Verify enablement configuration for a general app (Notes)",
 		T_META_TAG_VM_NOT_ELIGIBLE)
 {
-	bool should_defer_large = get_device_should_defer_large();
+	bool should_defer_large = false;
+#if MALLOC_TARGET_IOS_ONLY
+	// If an iOS device has >=6GB of memory, the enablement configuration
+	// should have defer_large set to "true"
+	uint64_t memsize = platform_hw_memsize();
+	const uint64_t defer_large_ios_bytes_memsize = 6 * 1073741824ULL;
+	T_LOG("Device memsize: %"PRIu64", defer_large_ios_bytes_memsize: %"PRIu64"",
+			memsize, defer_large_ios_bytes_memsize);
+	if (memsize >= defer_large_ios_bytes_memsize) {
+		should_defer_large = true;
+	}
+#endif
 
 	struct enablement_configuration configuration = (enablement_configuration) {
+#if TARGET_OS_OSX
+		.should_have_xzones = false,
+#else
 		.should_have_xzones = true,
+#endif // TARGET_OS_OSX
 		.guards_enabled = false,
 		.thread_cache_enabled = true,
+		.batch_size = 0,
 #if TARGET_OS_VISION
-		.batch_size = 0,
 		.ptr_bucket_count = 4,
-#elif TARGET_OS_OSX
-		.batch_size = 10,
-		.ptr_bucket_count = 4,
-#else
-		.batch_size = 0,
+#else
 		.ptr_bucket_count = 2,
-#endif
+#endif // TARGET_OS_VISION
 		.segment_group_ids_count = XZM_SEGMENT_GROUP_IDS_COUNT,
-#if TARGET_OS_OSX
-		.segment_group_count = get_ncpuclusters() *
-				(XZM_SEGMENT_GROUP_IDS_COUNT + ALLOCATION_FRONT_EXTRA),
-		.defer_tiny = true,
-		.defer_small = true,
-		.defer_large = true,
-#else
-		.segment_group_count = 1 * (XZM_SEGMENT_GROUP_IDS_COUNT +
-				ALLOCATION_FRONT_EXTRA),
+		.segment_group_count = 1 * XZM_SEGMENT_GROUP_IDS_COUNT,
 		.defer_tiny = false,
 		.defer_small = false,
 		.defer_large = should_defer_large,
-#endif // TARGET_OS_OSX
 	 };
 
 #if TARGET_OS_OSX
 	// Launch the Notes app on macOS
 	char *launch_notes_args[] = {"/System/Applications/Notes.app/Contents/MacOS/Notes", NULL};
-	pid_t notes_pid = spawn_process(launch_notes_args, NULL);
+	pid_t notes_pid = spawn_process(launch_notes_args);
 
 	// On macOS, we expect a random app process to not use xzone malloc
 	enablement_configuration_process_checks(
@@ -740,105 +651,11 @@
 #endif // TARGET_OS_OSX
 }
 
-#endif // !TARGET_OS_WATCH
-
-T_DECL(xzone_enabled_hardened_heap_entitlement_space_efficient,
-		"Verify enablement configuration for hardened-heap entitled process"
-		" (SpaceEfficient configuration)",
-		T_META_TAG_VM_NOT_ELIGIBLE)
-{
-	char *spawn_hardened_heap_args[] = {
-		"/AppleInternal/Tests/libmalloc/assets/hardened_heap_test_tool",
-		NULL,
-	};
-
-#if TARGET_OS_OSX
-	// SpaceEfficient is not the default on macOS
-	char *envp[] = {
-		"MallocSpaceEfficient=1",
-		NULL,
-	};
-#else
-	// SpaceEfficient is the default on other platforms
-	char **envp = NULL;
-#endif
-
-	pid_t pid = spawn_process(spawn_hardened_heap_args, envp);
-	bool space_efficient = true;
-
-	security_critical_configuration_checks_with_space_efficiency(
-			"hardened_heap_test_tool", space_efficient);
-
-	terminate_process(pid);
-}
-
-#if !TARGET_OS_WATCH
-
-T_DECL(xzone_enabled_hardened_heap_entitlement_non_space_efficient,
-		"Verify enablement configuration for hardened-heap entitled process"
-		" (non-SpaceEfficient configuration)",
-		T_META_TAG_VM_NOT_ELIGIBLE)
-{
-	char *spawn_hardened_heap_args[] = {
-		"/AppleInternal/Tests/libmalloc/assets/hardened_heap_test_tool",
-		NULL,
-	};
-
-#if TARGET_OS_OSX
-	// Non-SpaceEfficient is the default on macOS
-	char **envp = NULL;
-#else
-	// Non-SpaceEfficient requires nano envvar on other platforms, and
-	// MallocLargeCache on iOS
-	char *envp[] = {
-		"MallocNanoZone=1",
-#if MALLOC_TARGET_IOS_ONLY
-		"MallocLargeCache=1",
-#endif
-		NULL,
-	};
-#endif
-
-	pid_t pid = spawn_process(spawn_hardened_heap_args, envp);
-	bool space_efficient = false;
-
-	security_critical_configuration_checks_with_space_efficiency(
-			"hardened_heap_test_tool", space_efficient);
-
-	terminate_process(pid);
-}
-
-#endif // !TARGET_OS_WATCH
-
-#if MALLOC_TARGET_IOS_ONLY
-
-T_DECL(xzone_enabled_hardened_browser_entitlement,
-		"Verify enablement configuration for hardened-browser entitled process",
-		T_META_TAG_VM_NOT_ELIGIBLE)
-{
-	char *spawn_hardened_browser_args[] = {
-		"/AppleInternal/Tests/libmalloc/assets/hardened_browser_test_tool",
-		NULL,
-	};
-
-	pid_t pid = spawn_process(spawn_hardened_browser_args, NULL);
-
-	// The hardened-browser configuration is always SpaceEfficient
-	bool space_efficient = true;
-	security_critical_configuration_checks_with_space_efficiency(
-			"hardened_browser_test_tool", space_efficient);
-
-	terminate_process(pid);
-}
-
-#endif // MALLOC_TARGET_IOS_ONLY
-
 #else // CONFIG_XZONE_MALLOC && (MALLOC_TARGET_IOS_ONLY || TARGET_OS_OSX ||
-// TARGET_OS_VISION || TARGET_OS_WATCH)
-T_DECL(skip_json_printer_tests, "Skip printer tests",
-		T_META_TAG_VM_PREFERRED, T_META_TAG_NO_ALLOCATOR_OVERRIDE)
-{
-	T_SKIP("Nothing to test without xzone malloc");
+// TARGET_OS_VISION)
+T_DECL(skip_json_printer_tests, "Skip printer tests")
+{
+	T_SKIP("Nothing to test without xzone malloc on ios/macos/visionos");
 }
 #endif // CONFIG_XZONE_MALLOC && (MALLOC_TARGET_IOS_ONLY || TARGET_OS_OSX ||
 // TARGET_OS_VISION)