blob: 2791c53038c181ca7047b8b58119197df1685bb2 [file] [log] [blame]
Thiébaud Weksteenf24b4572021-11-26 09:12:41 +11001# Copyright 2021 The Android Open Source Project
2#
3# Licensed under the Apache License, Version 2.0 (the "License");
4# you may not use this file except in compliance with the License.
5# You may obtain a copy of the License at
6#
7# http://www.apache.org/licenses/LICENSE-2.0
8#
9# Unless required by applicable law or agreed to in writing, software
10# distributed under the License is distributed on an "AS IS" BASIS,
11# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12# See the License for the specific language governing permissions and
13# limitations under the License.
14
Dan Cashman91d398d2017-09-26 12:58:29 -070015from optparse import OptionParser
16from optparse import Option, OptionValueError
17import os
Inseob Kim4912a242022-07-25 11:30:02 +090018import pkgutil
Dan Cashman91d398d2017-09-26 12:58:29 -070019import policy
20import re
Inseob Kim4912a242022-07-25 11:30:02 +090021import shutil
Dan Cashman91d398d2017-09-26 12:58:29 -070022import sys
Inseob Kim4912a242022-07-25 11:30:02 +090023import tempfile
Inseob Kim3a9ac6f2022-07-19 14:27:36 +090024
25SHARED_LIB_EXTENSION = '.dylib' if sys.platform == 'darwin' else '.so'
Dan Cashman91d398d2017-09-26 12:58:29 -070026
27#############################################################
28# Tests
29#############################################################
30def TestDataTypeViolations(pol):
31return pol.AssertPathTypesHaveAttr(["/data/"], [], "data_file_type")
32
Nick Kralevichdab131b2018-10-04 11:24:00 -070033def TestSystemTypeViolations(pol):
Steven Morelanda01338d2020-04-27 15:54:54 -070034partitions = ["/system/", "/system_ext/", "/product/"]
35exceptions = [
36# devices before treble don't have a vendor partition
37"/system/vendor/",
38
39# overlay files are mounted over vendor
40"/product/overlay/",
41"/product/vendor_overlay/",
42"/system/overlay/",
43"/system/product/overlay/",
44"/system/product/vendor_overlay/",
45"/system/system_ext/overlay/",
46"/system_ext/overlay/",
Inseob Kim1b6a1292024-08-27 17:46:27 +090047
48# adb_keys_file hasn't been a system_file_type
49"/product/etc/security/adb_keys",
50"/system/product/etc/security/adb_keys",
Steven Morelanda01338d2020-04-27 15:54:54 -070051]
52
53return pol.AssertPathTypesHaveAttr(partitions, exceptions, "system_file_type")
Nick Kralevich5e372712018-09-27 10:21:37 -070054
Maciej Żenczykowskib13921c2022-05-21 05:03:29 -070055def TestBpffsTypeViolations(pol):
56return pol.AssertGenfsFilesystemTypesHaveAttr("bpf", "bpffs_type")
57
Tri Vo4c80c2c2018-03-29 03:42:47 +000058def TestProcTypeViolations(pol):
59return pol.AssertGenfsFilesystemTypesHaveAttr("proc", "proc_type")
60
Dan Cashman91d398d2017-09-26 12:58:29 -070061def TestSysfsTypeViolations(pol):
Jeff Vander Stoep1b828442018-03-21 17:27:20 -070062ret = pol.AssertGenfsFilesystemTypesHaveAttr("sysfs", "sysfs_type")
63ret += pol.AssertPathTypesHaveAttr(["/sys/"], ["/sys/kernel/debug/",
Dan Cashman91d398d2017-09-26 12:58:29 -070064"/sys/kernel/tracing"], "sysfs_type")
Jeff Vander Stoep1b828442018-03-21 17:27:20 -070065return ret
Dan Cashman91d398d2017-09-26 12:58:29 -070066
67def TestDebugfsTypeViolations(pol):
Jeff Vander Stoep1b828442018-03-21 17:27:20 -070068ret = pol.AssertGenfsFilesystemTypesHaveAttr("debugfs", "debugfs_type")
Jeff Vander Stoep1b828442018-03-21 17:27:20 -070069ret += pol.AssertPathTypesHaveAttr(["/sys/kernel/debug/",
Dan Cashman91d398d2017-09-26 12:58:29 -070070"/sys/kernel/tracing"], [], "debugfs_type")
Jeff Vander Stoep1b828442018-03-21 17:27:20 -070071return ret
Tri Vo30c3c2a2018-01-10 11:04:06 -080072
Hridya Valsarajuedccaa82021-05-14 14:01:11 -070073def TestTracefsTypeViolations(pol):
74ret = pol.AssertGenfsFilesystemTypesHaveAttr("tracefs", "tracefs_type")
75ret += pol.AssertPathTypesHaveAttr(["/sys/kernel/tracing"], [], "tracefs_type")
76ret += pol.AssertPathTypesDoNotHaveAttr(["/sys/kernel/debug"],
77["/sys/kernel/debug/tracing"], "tracefs_type",
78[])
79return ret
80
Tri Vo30c3c2a2018-01-10 11:04:06 -080081def TestVendorTypeViolations(pol):
Steven Morelanda01338d2020-04-27 15:54:54 -070082partitions = ["/vendor/", "/odm/"]
83exceptions = [
84"/vendor/etc/selinux/",
85"/vendor/odm/etc/selinux/",
86"/odm/etc/selinux/",
87]
88return pol.AssertPathTypesHaveAttr(partitions, exceptions, "vendor_file_type")
Tri Vo30c3c2a2018-01-10 11:04:06 -080089
Jeff Vander Stoepccf965e2018-01-24 07:01:13 -080090def TestCoreDataTypeViolations(pol):
Inseob Kimeb0d40a2023-09-04 19:02:53 +090091ret = pol.AssertPathTypesHaveAttr(["/data/"], ["/data/vendor",
Jeff Vander Stoep370a52f2018-02-08 09:54:59 -080092"/data/vendor_ce", "/data/vendor_de"], "core_data_file_type")
Inseob Kimeb0d40a2023-09-04 19:02:53 +090093ret += pol.AssertPathTypesDoNotHaveAttr(["/data/vendor/", "/data/vendor_ce/",
94"/data/vendor_de/"], [], "core_data_file_type")
95return ret
Jeff Vander Stoepccf965e2018-01-24 07:01:13 -080096
Inseob Kim1b8b1f62020-10-23 15:16:11 +090097def TestPropertyTypeViolations(pol):
98return pol.AssertPropertyOwnersAreExclusive()
99
Alan Stokes668e74f2020-11-12 18:08:18 +0000100def TestAppDataTypeViolations(pol):
101# Types with the app_data_file_type should only be used for app data files
102# (/data/data/package.name etc) via seapp_contexts, and never applied
103# explicitly to other files.
104partitions = [
105"/data/",
106"/vendor/",
107"/odm/",
108"/product/",
109]
110exceptions = [
111# These are used for app data files for the corresponding user and
112# assorted other files.
113# TODO(b/172812577): Use different types for the different purposes
114"shell_data_file",
115"bluetooth_data_file",
116"nfc_data_file",
117"radio_data_file",
118]
119return pol.AssertPathTypesDoNotHaveAttr(partitions, [], "app_data_file_type",
120exceptions)
Hridya Valsaraju8c9cf622020-12-14 22:57:49 -0800121def TestDmaHeapDevTypeViolations(pol):
122return pol.AssertPathTypesHaveAttr(["/dev/dma_heap/"], [],
123"dmabuf_heap_device_type")
124
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900125def TestCoredomainViolations(test_policy):
126# verify that all domains launched from /system have the coredomain
127# attribute
128ret = ""
Alan Stokes668e74f2020-11-12 18:08:18 +0000129
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900130for d in test_policy.alldomains:
131domain = test_policy.alldomains[d]
132if domain.fromSystem and domain.fromVendor:
133ret += "The following domain is system and vendor: " + d + "\n"
134
135for domain in test_policy.alldomains.values():
136ret += domain.error
137
138violators = []
139for d in test_policy.alldomains:
140domain = test_policy.alldomains[d]
141if domain.fromSystem and "coredomain" not in domain.attributes:
142violators.append(d);
143if len(violators) > 0:
144ret += "The following domain(s) must be associated with the "
145ret += "\"coredomain\" attribute because they are executed off of "
146ret += "/system:\n"
147ret += " ".join(str(x) for x in sorted(violators)) + "\n"
148
149# verify that all domains launched form /vendor do not have the coredomain
150# attribute
151violators = []
152for d in test_policy.alldomains:
153domain = test_policy.alldomains[d]
154if domain.fromVendor and "coredomain" in domain.attributes:
155violators.append(d)
156if len(violators) > 0:
157ret += "The following domains must not be associated with the "
158ret += "\"coredomain\" attribute because they are executed off of "
159ret += "/vendor or /system/vendor:\n"
160ret += " ".join(str(x) for x in sorted(violators)) + "\n"
161
162return ret
163
164def TestViolatorAttribute(test_policy, attribute):
165# TODO(b/113124961): re-enable once all violator attributes are removed.
166return ""
167
168# ret = ""
169# return ret
170
171# violators = test_policy.DomainsWithAttribute(attribute)
172# if len(violators) > 0:
173# ret += "SELinux: The following domains violate the Treble ban "
174# ret += "against use of the " + attribute + " attribute: "
175# ret += " ".join(str(x) for x in sorted(violators)) + "\n"
176# return ret
177
178def TestViolatorAttributes(test_policy):
179ret = ""
180ret += TestViolatorAttribute(test_policy, "socket_between_core_and_vendor_violators")
181ret += TestViolatorAttribute(test_policy, "vendor_executes_system_violators")
182return ret
183
184def TestIsolatedAttributeConsistency(test_policy):
185permissionAllowList = {
186# access given from technical_debt.cil
187"codec2_config_prop" : ["file"],
188"device_config_nnapi_native_prop":["file"],
Sandeep Bandaru702797d2024-09-20 14:55:25 +0000189"gpu_device": ["dir"],
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900190"hal_allocator_default":["binder", "fd"],
191"hal_codec2": ["binder", "fd"],
192"hal_codec2_hwservice":["hwservice_manager"],
193"hal_graphics_allocator": ["binder", "fd"],
194"hal_graphics_allocator_service":["service_manager"],
195"hal_graphics_allocator_hwservice":["hwservice_manager"],
196"hal_graphics_allocator_server":["binder", "service_manager"],
197"hal_graphics_mapper_hwservice":["hwservice_manager"],
Jooyung Han952673d2024-01-31 09:32:48 +0900198"hal_graphics_mapper_service":["service_manager"],
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900199"hal_neuralnetworks": ["binder", "fd"],
200"hal_neuralnetworks_service": ["service_manager"],
201"hal_neuralnetworks_hwservice":["hwservice_manager"],
202"hal_omx_hwservice":["hwservice_manager"],
203"hidl_allocator_hwservice":["hwservice_manager"],
204"hidl_manager_hwservice":["hwservice_manager"],
205"hidl_memory_hwservice":["hwservice_manager"],
206"hidl_token_hwservice":["hwservice_manager"],
207"hwservicemanager":["binder"],
208"hwservicemanager_prop":["file"],
209"mediacodec":["binder", "fd"],
210"mediaswcodec":["binder", "fd"],
211"media_variant_prop":["file"],
212"nnapi_ext_deny_product_prop":["file"],
213"servicemanager":["fd"],
Sandeep Bandaru702797d2024-09-20 14:55:25 +0000214"sysfs_gpu": ["file"],
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900215"toolbox_exec": ["file"],
216# extra types being granted to isolated_compute_app
217"isolated_compute_allowed":["service_manager", "chr_file"],
218}
219
220def resolveHalServerSubtype(target):
221# permission given as a client in technical_debt.cil
222hal_server_attributes = [
223"hal_codec2_server",
224"hal_graphics_allocator_server",
225"hal_neuralnetworks_server"]
226
227for attr in hal_server_attributes:
228if target in test_policy.pol.QueryTypeAttribute(Type=attr, IsAttr=True):
229return attr.rsplit("_", 1)[0]
230return target
231
232def checkIsolatedComputeAllowed(tctx, tclass):
233# check if the permission is in isolated_compute_allowed
234allowedMemberTypes = test_policy.pol.QueryTypeAttribute(Type="isolated_compute_allowed_service", IsAttr=True) \
235.union(test_policy.pol.QueryTypeAttribute(Type="isolated_compute_allowed_device", IsAttr=True))
236return tctx in allowedMemberTypes and tclass in permissionAllowList["isolated_compute_allowed"]
237
238def checkPermissions(permissions):
239violated_permissions = []
240for perm in permissions:
241tctx, tclass, p = perm.split(":")
242tctx = resolveHalServerSubtype(tctx)
243# check unwanted permissions
244if not checkIsolatedComputeAllowed(tctx, tclass) and \
245( tctx not in permissionAllowList \
246or tclass not in permissionAllowList[tctx] \
247or ( p == "write") \
248or ( p == "rw_file_perms") ):
249violated_permissions += [perm]
250return violated_permissions
251
252ret = ""
253
254isolatedMemberTypes = test_policy.pol.QueryTypeAttribute(Type="isolated_app_all", IsAttr=True)
255baseRules = test_policy.pol.QueryExpandedTERule(scontext=["isolated_app"])
256basePermissionSet = set([":".join([rule.tctx, rule.tclass, perm])
257for rule in baseRules for perm in rule.perms])
258for subType in isolatedMemberTypes:
259if subType == "isolated_app" : continue
260currentTypeRule = test_policy.pol.QueryExpandedTERule(scontext=[subType])
261typePermissionSet = set([":".join([rule.tctx, rule.tclass, perm])
262for rule in currentTypeRule for perm in rule.perms
263if not rule.tctx in [subType, subType + "_userfaultfd"]])
264deltaPermissionSet = typePermissionSet.difference(basePermissionSet)
265violated_permissions = checkPermissions(list(deltaPermissionSet))
266for perm in violated_permissions:
267ret += "allow %s %s:%s %s \n" % (subType, *perm.split(":"))
268
269if ret:
270ret = ("Found prohibited permission granted for isolated like types. " + \
271"Please replace your allow statements that involve \"-isolated_app\" with " + \
272"\"-isolated_app_all\". Violations are shown as the following: \n") + ret
273return ret
Inseob Kim1b8b1f62020-10-23 15:16:11 +0900274
Inseob Kim3a9d91c2023-09-27 17:39:07 +0900275def TestDevTypeViolations(pol):
276exceptions = [
277"/dev/socket",
278]
279exceptionTypes = [
280"boringssl_self_test_marker", # /dev/boringssl/selftest
281"cgroup_rc_file", # /dev/cgroup.rc
282"dev_cpu_variant", # /dev/cpu_variant:{arch}
283"fscklogs", # /dev/fscklogs
284"properties_serial", # /dev/__properties__/properties_serial
285"property_info", # /dev/__properties__/property_info
286"runtime_event_log_tags_file", # /dev/event-log-tags
287]
288return pol.AssertPathTypesHaveAttr(["/dev"], exceptions,
289"dev_type", exceptionTypes)
290
Dan Cashman91d398d2017-09-26 12:58:29 -0700291###
292# extend OptionParser to allow the same option flag to be used multiple times.
293# This is used to allow multiple file_contexts files and tests to be
294# specified.
295#
296class MultipleOption(Option):
297ACTIONS = Option.ACTIONS + ("extend",)
298STORE_ACTIONS = Option.STORE_ACTIONS + ("extend",)
299TYPED_ACTIONS = Option.TYPED_ACTIONS + ("extend",)
300ALWAYS_TYPED_ACTIONS = Option.ALWAYS_TYPED_ACTIONS + ("extend",)
301
302def take_action(self, action, dest, opt, value, values, parser):
303if action == "extend":
304values.ensure_value(dest, []).append(value)
305else:
306Option.take_action(self, action, dest, opt, value, values, parser)
307
Thiébaud Weksteen70cf2cd2024-05-14 13:36:34 +1000308TEST_NAMES = [ name for name in dir() if name.startswith('Test') ]
Dan Cashman91d398d2017-09-26 12:58:29 -0700309
Inseob Kim4912a242022-07-25 11:30:02 +0900310def do_main(libpath):
311"""
312Args:
313libpath: string, path to libsepolwrap.so
314"""
Inseob Kim6fa8efd2021-12-29 13:56:14 +0900315usage = "sepolicy_tests -f vendor_file_contexts -f "
Dan Cashman91d398d2017-09-26 12:58:29 -0700316usage +="plat_file_contexts -p policy [--test test] [--help]"
317parser = OptionParser(option_class=MultipleOption, usage=usage)
318parser.add_option("-f", "--file_contexts", dest="file_contexts",
319metavar="FILE", action="extend", type="string")
320parser.add_option("-p", "--policy", dest="policy", metavar="FILE")
Dan Cashman91d398d2017-09-26 12:58:29 -0700321parser.add_option("-t", "--test", dest="test", action="extend",
Thiébaud Weksteen70cf2cd2024-05-14 13:36:34 +1000322help="Test options include "+str(TEST_NAMES))
Dan Cashman91d398d2017-09-26 12:58:29 -0700323
324(options, args) = parser.parse_args()
325
Dan Cashman91d398d2017-09-26 12:58:29 -0700326if not options.policy:
327sys.exit("Must specify monolithic policy file\n" + parser.usage)
328if not os.path.exists(options.policy):
329sys.exit("Error: policy file " + options.policy + " does not exist\n"
330+ parser.usage)
331
332if not options.file_contexts:
333sys.exit("Error: Must specify file_contexts file(s)\n" + parser.usage)
334for f in options.file_contexts:
335if not os.path.exists(f):
336sys.exit("Error: File_contexts file " + f + " does not exist\n" +
337parser.usage)
338
Inseob Kim6fa8efd2021-12-29 13:56:14 +0900339pol = policy.Policy(options.policy, options.file_contexts, libpath)
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900340test_policy = policy.TestPolicy()
341test_policy.setup(pol)
Dan Cashman91d398d2017-09-26 12:58:29 -0700342
343results = ""
344# If an individual test is not specified, run all tests.
Maciej Żenczykowskib13921c2022-05-21 05:03:29 -0700345if options.test is None or "TestBpffsTypeViolations" in options.test:
346results += TestBpffsTypeViolations(pol)
Jeff Vander Stoep3ca843a2017-10-04 09:42:29 -0700347if options.test is None or "TestDataTypeViolations" in options.test:
Dan Cashman91d398d2017-09-26 12:58:29 -0700348results += TestDataTypeViolations(pol)
Tri Vo4c80c2c2018-03-29 03:42:47 +0000349if options.test is None or "TestProcTypeViolations" in options.test:
350results += TestProcTypeViolations(pol)
Jeff Vander Stoep3ca843a2017-10-04 09:42:29 -0700351if options.test is None or "TestSysfsTypeViolations" in options.test:
Dan Cashman91d398d2017-09-26 12:58:29 -0700352results += TestSysfsTypeViolations(pol)
Nick Kralevichdab131b2018-10-04 11:24:00 -0700353if options.test is None or "TestSystemTypeViolations" in options.test:
354results += TestSystemTypeViolations(pol)
Jeff Vander Stoep3ca843a2017-10-04 09:42:29 -0700355if options.test is None or "TestDebugfsTypeViolations" in options.test:
Dan Cashman91d398d2017-09-26 12:58:29 -0700356results += TestDebugfsTypeViolations(pol)
Hridya Valsarajuedccaa82021-05-14 14:01:11 -0700357if options.test is None or "TestTracefsTypeViolations" in options.test:
358results += TestTracefsTypeViolations(pol)
Tri Vo30c3c2a2018-01-10 11:04:06 -0800359if options.test is None or "TestVendorTypeViolations" in options.test:
360results += TestVendorTypeViolations(pol)
Jeff Vander Stoepccf965e2018-01-24 07:01:13 -0800361if options.test is None or "TestCoreDataTypeViolations" in options.test:
362results += TestCoreDataTypeViolations(pol)
Inseob Kim1b8b1f62020-10-23 15:16:11 +0900363if options.test is None or "TestPropertyTypeViolations" in options.test:
364results += TestPropertyTypeViolations(pol)
Alan Stokes668e74f2020-11-12 18:08:18 +0000365if options.test is None or "TestAppDataTypeViolations" in options.test:
366results += TestAppDataTypeViolations(pol)
Hridya Valsaraju8c9cf622020-12-14 22:57:49 -0800367if options.test is None or "TestDmaHeapDevTypeViolations" in options.test:
368results += TestDmaHeapDevTypeViolations(pol)
Inseob Kimeb0d40a2023-09-04 19:02:53 +0900369if options.test is None or "TestCoredomainViolations" in options.test:
370results += TestCoredomainViolations(test_policy)
371if options.test is None or "TestViolatorAttributes" in options.test:
372results += TestViolatorAttributes(test_policy)
373if options.test is None or "TestIsolatedAttributeConsistency" in options.test:
374results += TestIsolatedAttributeConsistency(test_policy)
Dan Cashman91d398d2017-09-26 12:58:29 -0700375
Inseob Kim3a9d91c2023-09-27 17:39:07 +0900376# dev type test won't be run as default
377if options.test and "TestDevTypeViolations" in options.test:
378results += TestDevTypeViolations(pol)
379
Dan Cashman91d398d2017-09-26 12:58:29 -0700380if len(results) > 0:
381sys.exit(results)
Inseob Kim4912a242022-07-25 11:30:02 +0900382
383if __name__ == '__main__':
384temp_dir = tempfile.mkdtemp()
385try:
386libname = "libsepolwrap" + SHARED_LIB_EXTENSION
387libpath = os.path.join(temp_dir, libname)
388with open(libpath, "wb") as f:
389blob = pkgutil.get_data("sepolicy_tests", libname)
390if not blob:
391sys.exit("Error: libsepolwrap does not exist. Is this binary corrupted?\n")
392f.write(blob)
393do_main(libpath)
394finally:
395shutil.rmtree(temp_dir)