blob: 57373d5b9b48bf487e166732e3857a04ee0061e8 [file] [log] [blame]
Steven Moreland65b3fd92017-12-06 14:18:35 -08001// Copyright 2017 Google Inc. All rights reserved.
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
15package android
16
17import (
Liz Kammera3d79152021-10-28 18:14:04 -040018 "fmt"
Steven Moreland65b3fd92017-12-06 14:18:35 -080019 "path/filepath"
20 "reflect"
Anton Hansson45376402020-04-09 14:18:21 +010021 "regexp"
Steven Moreland65b3fd92017-12-06 14:18:35 -080022 "strconv"
23 "strings"
24
25 "github.com/google/blueprint/proptools"
26)
27
28// "neverallow" rules for the build system.
29//
30// This allows things which aren't related to the build system and are enforced
Joe Onoratob4638c12021-10-27 15:47:06 -070031// against assumptions, in progress code refactors, or policy to be expressed in a
Steven Moreland65b3fd92017-12-06 14:18:35 -080032// straightforward away disjoint from implementations and tests which should
33// work regardless of these restrictions.
34//
35// A module is disallowed if all of the following are true:
Paul Duffin730f2a52019-06-27 14:08:51 +010036// - it is in one of the "In" paths
37// - it is not in one of the "NotIn" paths
38// - it has all "With" properties matched
Steven Moreland65b3fd92017-12-06 14:18:35 -080039// - - values are matched in their entirety
40// - - nil is interpreted as an empty string
41// - - nested properties are separated with a '.'
42// - - if the property is a list, any of the values in the list being matches
43// counts as a match
Paul Duffin730f2a52019-06-27 14:08:51 +010044// - it has none of the "Without" properties matched (same rules as above)
Steven Moreland65b3fd92017-12-06 14:18:35 -080045
Paul Duffin45338f02021-03-30 23:07:52 +010046func registerNeverallowMutator(ctx RegisterMutatorsContext) {
Colin Cross8a962802024-10-09 15:29:27 -070047 ctx.BottomUp("neverallow", neverallowMutator)
Steven Moreland65b3fd92017-12-06 14:18:35 -080048}
49
Paul Duffin730f2a52019-06-27 14:08:51 +010050var neverallows = []Rule{}
Steven Moreland65b3fd92017-12-06 14:18:35 -080051
Paul Duffin730f2a52019-06-27 14:08:51 +010052func init() {
Paul Duffinc8111702019-07-22 12:13:55 +010053 AddNeverAllowRules(createIncludeDirsRules()...)
Paul Duffin730f2a52019-06-27 14:08:51 +010054 AddNeverAllowRules(createTrebleRules()...)
Paul Duffin730f2a52019-06-27 14:08:51 +010055 AddNeverAllowRules(createJavaDeviceForHostRules()...)
Colin Crossc511bc52020-04-07 16:50:32 +000056 AddNeverAllowRules(createCcSdkVariantRules()...)
David Srbeckye033cba2020-05-20 22:20:28 +010057 AddNeverAllowRules(createUncompressDexRules()...)
Inseob Kim800d1142021-06-14 12:03:51 +090058 AddNeverAllowRules(createInitFirstStageRules()...)
Jiyong Park3c306f32022-04-05 15:29:53 +090059 AddNeverAllowRules(createProhibitFrameworkAccessRules()...)
Alan Stokes73feba32022-11-14 12:21:24 +000060 AddNeverAllowRules(createCcStubsRule())
Mark Whitea15790a2023-08-22 21:28:11 +000061 AddNeverAllowRules(createProhibitHeaderOnlyRule())
Steven Moreland0db999c2024-09-03 22:06:24 +000062 AddNeverAllowRules(createLimitNdkExportRule()...)
Inseob Kim76e19852024-10-10 17:57:22 +090063 AddNeverAllowRules(createLimitDirgroupRule()...)
Jihoon Kang0d545b82024-10-11 00:21:57 +000064 AddNeverAllowRules(createFilesystemIsAutoGeneratedRule())
Luca Stefani50098f72024-10-12 17:55:31 +020065 AddNeverAllowRules(createKotlinPluginRule()...)
Neil Fullerdf5f3562018-10-21 17:19:10 +010066}
Steven Moreland65b3fd92017-12-06 14:18:35 -080067
Paul Duffin730f2a52019-06-27 14:08:51 +010068// Add a NeverAllow rule to the set of rules to apply.
69func AddNeverAllowRules(rules ...Rule) {
70 neverallows = append(neverallows, rules...)
71}
72
Sam Delmerico46d08b42022-11-15 15:51:04 -050073var (
74 neverallowNotInIncludeDir = []string{
Paul Duffinc8111702019-07-22 12:13:55 +010075 "art",
Orion Hodson6341f012019-11-06 13:39:46 +000076 "art/libnativebridge",
77 "art/libnativeloader",
Paul Duffinc8111702019-07-22 12:13:55 +010078 "libcore",
79 "libnativehelper",
80 "external/apache-harmony",
81 "external/apache-xml",
82 "external/boringssl",
83 "external/bouncycastle",
84 "external/conscrypt",
85 "external/icu",
86 "external/okhttp",
87 "external/vixl",
88 "external/wycheproof",
Paul Duffinc8111702019-07-22 12:13:55 +010089 }
Sam Delmerico46d08b42022-11-15 15:51:04 -050090 neverallowNoUseIncludeDir = []string{
Steven Morelandf36a3ac2021-04-27 18:03:14 +000091 "frameworks/av/apex",
92 "frameworks/av/tools",
93 "frameworks/native/cmds",
94 "system/apex",
95 "system/bpf",
96 "system/gatekeeper",
97 "system/hwservicemanager",
98 "system/libbase",
Steven Moreland8fc8dbf2021-04-27 02:31:07 +000099 "system/libfmq",
Steven Morelandf36a3ac2021-04-27 18:03:14 +0000100 "system/libvintf",
Steven Moreland8fc8dbf2021-04-27 02:31:07 +0000101 }
Sam Delmerico46d08b42022-11-15 15:51:04 -0500102)
Paul Duffinc8111702019-07-22 12:13:55 +0100103
Sam Delmerico46d08b42022-11-15 15:51:04 -0500104func createIncludeDirsRules() []Rule {
105 rules := make([]Rule, 0, len(neverallowNotInIncludeDir)+len(neverallowNoUseIncludeDir))
Steven Moreland8fc8dbf2021-04-27 02:31:07 +0000106
Sam Delmerico46d08b42022-11-15 15:51:04 -0500107 for _, path := range neverallowNotInIncludeDir {
Paul Duffinc8111702019-07-22 12:13:55 +0100108 rule :=
109 NeverAllow().
110 WithMatcher("include_dirs", StartsWith(path+"/")).
111 Because("include_dirs is deprecated, all usages of '" + path + "' have been migrated" +
112 " to use alternate mechanisms and so can no longer be used.")
113
114 rules = append(rules, rule)
115 }
116
Sam Delmerico46d08b42022-11-15 15:51:04 -0500117 for _, path := range neverallowNoUseIncludeDir {
Steven Moreland8fc8dbf2021-04-27 02:31:07 +0000118 rule := NeverAllow().In(path+"/").WithMatcher("include_dirs", isSetMatcherInstance).
119 Because("include_dirs is deprecated, all usages of them in '" + path + "' have been migrated" +
120 " to use alternate mechanisms and so can no longer be used.")
121 rules = append(rules, rule)
122 }
123
Paul Duffinc8111702019-07-22 12:13:55 +0100124 return rules
125}
126
Paul Duffin730f2a52019-06-27 14:08:51 +0100127func createTrebleRules() []Rule {
128 return []Rule{
129 NeverAllow().
130 In("vendor", "device").
131 With("vndk.enabled", "true").
132 Without("vendor", "true").
Justin Yun0ecf0b22020-02-28 15:07:59 +0900133 Without("product_specific", "true").
Paul Duffin730f2a52019-06-27 14:08:51 +0100134 Because("the VNDK can never contain a library that is device dependent."),
135 NeverAllow().
136 With("vndk.enabled", "true").
137 Without("vendor", "true").
138 Without("owner", "").
139 Because("a VNDK module can never have an owner."),
Steven Moreland65b3fd92017-12-06 14:18:35 -0800140
Neil Fullerdf5f3562018-10-21 17:19:10 +0100141 // TODO(b/67974785): always enforce the manifest
Paul Duffin730f2a52019-06-27 14:08:51 +0100142 NeverAllow().
Steven Moreland51ce4f62020-02-10 17:21:32 -0800143 Without("name", "libhidlbase-combined-impl").
144 Without("name", "libhidlbase").
Paul Duffin730f2a52019-06-27 14:08:51 +0100145 With("product_variables.enforce_vintf_manifest.cflags", "*").
146 Because("manifest enforcement should be independent of ."),
Neil Fullerdf5f3562018-10-21 17:19:10 +0100147
148 // TODO(b/67975799): vendor code should always use /vendor/bin/sh
Paul Duffin730f2a52019-06-27 14:08:51 +0100149 NeverAllow().
150 Without("name", "libc_bionic_ndk").
151 With("product_variables.treble_linker_namespaces.cflags", "*").
152 Because("nothing should care if linker namespaces are enabled or not"),
Neil Fullerdf5f3562018-10-21 17:19:10 +0100153
154 // Example:
Paul Duffin730f2a52019-06-27 14:08:51 +0100155 // *NeverAllow().with("Srcs", "main.cpp"))
Neil Fullerdf5f3562018-10-21 17:19:10 +0100156 }
157}
158
Paul Duffin730f2a52019-06-27 14:08:51 +0100159func createJavaDeviceForHostRules() []Rule {
Colin Cross440e0d02020-06-11 11:32:11 -0700160 javaDeviceForHostProjectsAllowedList := []string{
Dan Willemsen9fe14102021-07-13 21:52:04 -0700161 "development/build",
Colin Crossb5191a52019-04-11 14:07:38 -0700162 "external/guava",
Steve Elliott8053f822022-10-18 17:09:28 -0400163 "external/kotlinx.coroutines",
Colin Crossfd4f7432019-03-05 15:06:16 -0800164 "external/robolectric-shadows",
Rex Hoffman54641d22022-08-25 17:29:50 +0000165 "external/robolectric",
Makoto Onukib66bba32023-11-11 00:06:09 +0000166 "frameworks/base/ravenwood",
167 "frameworks/base/tools/hoststubgen",
Jerome Gaillard655ee022021-09-23 11:38:08 +0000168 "frameworks/layoutlib",
Colin Crossfd4f7432019-03-05 15:06:16 -0800169 }
170
Paul Duffin730f2a52019-06-27 14:08:51 +0100171 return []Rule{
172 NeverAllow().
Colin Cross440e0d02020-06-11 11:32:11 -0700173 NotIn(javaDeviceForHostProjectsAllowedList...).
Paul Duffin730f2a52019-06-27 14:08:51 +0100174 ModuleType("java_device_for_host", "java_host_for_device").
Colin Cross440e0d02020-06-11 11:32:11 -0700175 Because("java_device_for_host can only be used in allowed projects"),
Colin Crossfd4f7432019-03-05 15:06:16 -0800176 }
177}
178
Colin Crossc511bc52020-04-07 16:50:32 +0000179func createCcSdkVariantRules() []Rule {
Colin Cross440e0d02020-06-11 11:32:11 -0700180 sdkVersionOnlyAllowedList := []string{
Colin Crossc511bc52020-04-07 16:50:32 +0000181 // derive_sdk_prefer32 has stem: "derive_sdk" which conflicts with the derive_sdk.
182 // This sometimes works because the APEX modules that contain derive_sdk and
183 // derive_sdk_prefer32 suppress the platform installation rules, but fails when
184 // the APEX modules contain the SDK variant and the platform variant still exists.
Anton Hansson4b8e64b2020-05-27 18:25:23 +0100185 "packages/modules/SdkExtensions/derive_sdk",
Dan Alberte2054a92020-04-20 14:46:47 -0700186 // These are for apps and shouldn't be used by non-SDK variant modules.
187 "prebuilts/ndk",
Jiyong Parka574d532024-08-28 18:06:43 +0900188 "frameworks/native/libs/binder/ndk",
Dan Alberte2054a92020-04-20 14:46:47 -0700189 "tools/test/graphicsbenchmark/apps/sample_app",
190 "tools/test/graphicsbenchmark/functional_tests/java",
Dan Albert55576052020-04-20 14:46:47 -0700191 "vendor/xts/gts-tests/hostsidetests/gamedevicecert/apps/javatests",
Chang Li66d3cb72021-06-18 14:04:50 +0000192 "external/libtextclassifier/native",
Colin Crossc511bc52020-04-07 16:50:32 +0000193 }
194
Colin Cross440e0d02020-06-11 11:32:11 -0700195 platformVariantPropertiesAllowedList := []string{
Colin Crossc511bc52020-04-07 16:50:32 +0000196 // android_native_app_glue and libRSSupport use native_window.h but target old
197 // sdk versions (minimum and 9 respectively) where libnativewindow didn't exist,
198 // so they can't add libnativewindow to shared_libs to get the header directory
199 // for the platform variant. Allow them to use the platform variant
200 // property to set shared_libs.
201 "prebuilts/ndk",
202 "frameworks/rs",
203 }
204
205 return []Rule{
206 NeverAllow().
Colin Cross440e0d02020-06-11 11:32:11 -0700207 NotIn(sdkVersionOnlyAllowedList...).
Colin Crossc511bc52020-04-07 16:50:32 +0000208 WithMatcher("sdk_variant_only", isSetMatcherInstance).
Colin Cross440e0d02020-06-11 11:32:11 -0700209 Because("sdk_variant_only can only be used in allowed projects"),
Colin Crossc511bc52020-04-07 16:50:32 +0000210 NeverAllow().
Colin Cross440e0d02020-06-11 11:32:11 -0700211 NotIn(platformVariantPropertiesAllowedList...).
Colin Crossc511bc52020-04-07 16:50:32 +0000212 WithMatcher("platform.shared_libs", isSetMatcherInstance).
Colin Cross440e0d02020-06-11 11:32:11 -0700213 Because("platform variant properties can only be used in allowed projects"),
Colin Crossc511bc52020-04-07 16:50:32 +0000214 }
215}
216
Alan Stokes73feba32022-11-14 12:21:24 +0000217func createCcStubsRule() Rule {
218 ccStubsImplementationInstallableProjectsAllowedList := []string{
Jiyong Parkf6736c72024-07-22 11:22:35 +0900219 "packages/modules/Virtualization/libs/libvm_payload",
Alan Stokes73feba32022-11-14 12:21:24 +0000220 }
221
222 return NeverAllow().
223 NotIn(ccStubsImplementationInstallableProjectsAllowedList...).
224 WithMatcher("stubs.implementation_installable", isSetMatcherInstance).
225 Because("implementation_installable can only be used in allowed projects.")
226}
227
David Srbeckye033cba2020-05-20 22:20:28 +0100228func createUncompressDexRules() []Rule {
229 return []Rule{
230 NeverAllow().
231 NotIn("art").
232 WithMatcher("uncompress_dex", isSetMatcherInstance).
233 Because("uncompress_dex is only allowed for certain jars for test in art."),
234 }
235}
236
Inseob Kim800d1142021-06-14 12:03:51 +0900237func createInitFirstStageRules() []Rule {
238 return []Rule{
239 NeverAllow().
Nikita Ioffe11a9c2c2023-06-21 16:51:09 +0100240 Without("name", "init_first_stage_defaults").
Inseob Kim800d1142021-06-14 12:03:51 +0900241 Without("name", "init_first_stage").
Nikita Ioffe11a9c2c2023-06-21 16:51:09 +0100242 Without("name", "init_first_stage.microdroid").
Inseob Kim800d1142021-06-14 12:03:51 +0900243 With("install_in_root", "true").
Inseob Kimbe6a66d2024-07-11 15:43:44 +0900244 NotModuleType("prebuilt_root").
Inseob Kim800d1142021-06-14 12:03:51 +0900245 Because("install_in_root is only for init_first_stage."),
246 }
247}
248
Jiyong Park3c306f32022-04-05 15:29:53 +0900249func createProhibitFrameworkAccessRules() []Rule {
250 return []Rule{
251 NeverAllow().
252 With("libs", "framework").
253 WithoutMatcher("sdk_version", Regexp("(core_.*|^$)")).
254 Because("framework can't be used when building against SDK"),
255 }
256}
257
Mark Whitea15790a2023-08-22 21:28:11 +0000258func createProhibitHeaderOnlyRule() Rule {
259 return NeverAllow().
260 Without("name", "framework-minus-apex-headers").
261 With("headers_only", "true").
262 Because("headers_only can only be used for generating framework-minus-apex headers for non-updatable modules")
263}
264
Steven Moreland0db999c2024-09-03 22:06:24 +0000265func createLimitNdkExportRule() []Rule {
266 reason := "If the headers you're trying to export are meant to be a part of the NDK, they should be exposed by an ndk_headers module. If the headers shouldn't be a part of the NDK, the headers should instead be exposed from a separate `cc_library_headers` which consumers depend on."
267 // DO NOT ADD HERE - please consult danalbert@
268 // b/357711733
269 return []Rule{
270 NeverAllow().
271 NotIn("frameworks/native/libs/binder/ndk").
272 ModuleType("ndk_library").
273 WithMatcher("export_header_libs", isSetMatcherInstance).Because(reason),
274 NeverAllow().ModuleType("ndk_library").WithMatcher("export_generated_headers", isSetMatcherInstance).Because(reason),
275 NeverAllow().ModuleType("ndk_library").WithMatcher("export_include_dirs", isSetMatcherInstance).Because(reason),
276 NeverAllow().ModuleType("ndk_library").WithMatcher("export_shared_lib_headers", isSetMatcherInstance).Because(reason),
277 NeverAllow().ModuleType("ndk_library").WithMatcher("export_static_lib_headers", isSetMatcherInstance).Because(reason),
278 }
279}
280
Inseob Kim76e19852024-10-10 17:57:22 +0900281func createLimitDirgroupRule() []Rule {
282 reason := "dirgroup module and dir_srcs property of genrule is allowed only to Trusty build rule."
283 return []Rule{
284 NeverAllow().
285 ModuleType("dirgroup").
286 WithMatcher("visibility", NotInList([]string{"//trusty/vendor/google/aosp/scripts"})).Because(reason),
287 NeverAllow().
288 ModuleType("dirgroup").
289 Without("visibility", "//trusty/vendor/google/aosp/scripts").Because(reason),
290 NeverAllow().
291 ModuleType("genrule").
292 Without("name", "lk.elf.arm64").
293 Without("name", "lk.elf.x86_64").
294 WithMatcher("dir_srcs", isSetMatcherInstance).Because(reason),
295 }
296}
297
Jihoon Kang0d545b82024-10-11 00:21:57 +0000298func createFilesystemIsAutoGeneratedRule() Rule {
299 return NeverAllow().
300 NotIn("build/soong/fsgen").
301 ModuleType("filesystem", "android_system_image").
302 WithMatcher("is_auto_generated", isSetMatcherInstance).
303 Because("is_auto_generated property is only allowed for filesystem modules in build/soong/fsgen directory")
304}
305
Luca Stefani50098f72024-10-12 17:55:31 +0200306func createKotlinPluginRule() []Rule {
307 kotlinPluginProjectsAllowedList := []string{
308 // TODO: Migrate compose plugin to the bundled compiler plugin
309 // Actual path prebuilts/sdk/current/androidx/m2repository/androidx/compose/compiler/compiler-hosted
310 "prebuilts/sdk/current/androidx",
311 "external/kotlinc",
312 }
313
314 return []Rule{
315 NeverAllow().
316 NotIn(kotlinPluginProjectsAllowedList...).
317 ModuleType("kotlin_plugin").
318 Because("kotlin_plugin can only be used in allowed projects"),
319 }
320}
321
Steven Moreland65b3fd92017-12-06 14:18:35 -0800322func neverallowMutator(ctx BottomUpMutatorContext) {
323 m, ok := ctx.Module().(Module)
324 if !ok {
325 return
326 }
327
328 dir := ctx.ModuleDir() + "/"
329 properties := m.GetProperties()
330
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100331 osClass := ctx.Module().Target().Os.Class
332
Paul Duffin115445b2019-08-07 15:31:07 +0100333 for _, r := range neverallowRules(ctx.Config()) {
Paul Duffin730f2a52019-06-27 14:08:51 +0100334 n := r.(*rule)
Steven Moreland65b3fd92017-12-06 14:18:35 -0800335 if !n.appliesToPath(dir) {
336 continue
337 }
338
Colin Crossfd4f7432019-03-05 15:06:16 -0800339 if !n.appliesToModuleType(ctx.ModuleType()) {
340 continue
341 }
342
Cole Faust5b35cb92024-08-27 15:47:06 -0700343 if !n.appliesToProperties(ctx, properties) {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800344 continue
345 }
346
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100347 if !n.appliesToOsClass(osClass) {
348 continue
349 }
350
Paul Duffin35781882019-07-25 15:41:09 +0100351 if !n.appliesToDirectDeps(ctx) {
352 continue
353 }
354
Steven Moreland65b3fd92017-12-06 14:18:35 -0800355 ctx.ModuleErrorf("violates " + n.String())
356 }
357}
358
Paul Duffin73bf0542019-07-12 14:12:49 +0100359type ValueMatcher interface {
Anton Hanssone1b18362021-12-23 15:05:38 +0000360 Test(string) bool
Paul Duffin73bf0542019-07-12 14:12:49 +0100361 String() string
362}
363
364type equalMatcher struct {
365 expected string
366}
367
Anton Hanssone1b18362021-12-23 15:05:38 +0000368func (m *equalMatcher) Test(value string) bool {
Paul Duffin73bf0542019-07-12 14:12:49 +0100369 return m.expected == value
370}
371
372func (m *equalMatcher) String() string {
373 return "=" + m.expected
374}
375
376type anyMatcher struct {
377}
378
Anton Hanssone1b18362021-12-23 15:05:38 +0000379func (m *anyMatcher) Test(value string) bool {
Paul Duffin73bf0542019-07-12 14:12:49 +0100380 return true
381}
382
383func (m *anyMatcher) String() string {
384 return "=*"
385}
386
387var anyMatcherInstance = &anyMatcher{}
388
Paul Duffinc8111702019-07-22 12:13:55 +0100389type startsWithMatcher struct {
390 prefix string
391}
392
Anton Hanssone1b18362021-12-23 15:05:38 +0000393func (m *startsWithMatcher) Test(value string) bool {
Paul Duffinc8111702019-07-22 12:13:55 +0100394 return strings.HasPrefix(value, m.prefix)
395}
396
397func (m *startsWithMatcher) String() string {
398 return ".starts-with(" + m.prefix + ")"
399}
400
Anton Hansson45376402020-04-09 14:18:21 +0100401type regexMatcher struct {
402 re *regexp.Regexp
403}
404
Anton Hanssone1b18362021-12-23 15:05:38 +0000405func (m *regexMatcher) Test(value string) bool {
Anton Hansson45376402020-04-09 14:18:21 +0100406 return m.re.MatchString(value)
407}
408
409func (m *regexMatcher) String() string {
410 return ".regexp(" + m.re.String() + ")"
411}
412
Andrei Onea115e7e72020-06-05 21:14:03 +0100413type notInListMatcher struct {
414 allowed []string
415}
416
Anton Hanssone1b18362021-12-23 15:05:38 +0000417func (m *notInListMatcher) Test(value string) bool {
Andrei Onea115e7e72020-06-05 21:14:03 +0100418 return !InList(value, m.allowed)
419}
420
421func (m *notInListMatcher) String() string {
422 return ".not-in-list(" + strings.Join(m.allowed, ",") + ")"
423}
424
Colin Crossc511bc52020-04-07 16:50:32 +0000425type isSetMatcher struct{}
426
Anton Hanssone1b18362021-12-23 15:05:38 +0000427func (m *isSetMatcher) Test(value string) bool {
Colin Crossc511bc52020-04-07 16:50:32 +0000428 return value != ""
429}
430
431func (m *isSetMatcher) String() string {
432 return ".is-set"
433}
434
435var isSetMatcherInstance = &isSetMatcher{}
436
Steven Moreland65b3fd92017-12-06 14:18:35 -0800437type ruleProperty struct {
Paul Duffin73bf0542019-07-12 14:12:49 +0100438 fields []string // e.x.: Vndk.Enabled
439 matcher ValueMatcher
Steven Moreland65b3fd92017-12-06 14:18:35 -0800440}
441
Liz Kammera3d79152021-10-28 18:14:04 -0400442func (r *ruleProperty) String() string {
443 return fmt.Sprintf("%q matches: %s", strings.Join(r.fields, "."), r.matcher)
444}
445
446type ruleProperties []ruleProperty
447
448func (r ruleProperties) String() string {
449 var s []string
450 for _, r := range r {
451 s = append(s, r.String())
452 }
453 return strings.Join(s, " ")
454}
455
Paul Duffin730f2a52019-06-27 14:08:51 +0100456// A NeverAllow rule.
457type Rule interface {
458 In(path ...string) Rule
459
460 NotIn(path ...string) Rule
461
Paul Duffin35781882019-07-25 15:41:09 +0100462 InDirectDeps(deps ...string) Rule
463
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100464 WithOsClass(osClasses ...OsClass) Rule
465
Paul Duffin730f2a52019-06-27 14:08:51 +0100466 ModuleType(types ...string) Rule
467
468 NotModuleType(types ...string) Rule
469
470 With(properties, value string) Rule
471
Paul Duffinc8111702019-07-22 12:13:55 +0100472 WithMatcher(properties string, matcher ValueMatcher) Rule
473
Paul Duffin730f2a52019-06-27 14:08:51 +0100474 Without(properties, value string) Rule
475
Paul Duffinc8111702019-07-22 12:13:55 +0100476 WithoutMatcher(properties string, matcher ValueMatcher) Rule
477
Paul Duffin730f2a52019-06-27 14:08:51 +0100478 Because(reason string) Rule
479}
480
Steven Moreland65b3fd92017-12-06 14:18:35 -0800481type rule struct {
482 // User string for why this is a thing.
483 reason string
484
485 paths []string
486 unlessPaths []string
487
Paul Duffin35781882019-07-25 15:41:09 +0100488 directDeps map[string]bool
489
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100490 osClasses []OsClass
491
Colin Crossfd4f7432019-03-05 15:06:16 -0800492 moduleTypes []string
493 unlessModuleTypes []string
494
Liz Kammera3d79152021-10-28 18:14:04 -0400495 props ruleProperties
496 unlessProps ruleProperties
Andrei Onea115e7e72020-06-05 21:14:03 +0100497
498 onlyBootclasspathJar bool
Steven Moreland65b3fd92017-12-06 14:18:35 -0800499}
500
Paul Duffin730f2a52019-06-27 14:08:51 +0100501// Create a new NeverAllow rule.
502func NeverAllow() Rule {
Paul Duffin35781882019-07-25 15:41:09 +0100503 return &rule{directDeps: make(map[string]bool)}
Steven Moreland65b3fd92017-12-06 14:18:35 -0800504}
Colin Crossfd4f7432019-03-05 15:06:16 -0800505
Liz Kammera3d79152021-10-28 18:14:04 -0400506// In adds path(s) where this rule applies.
Paul Duffin730f2a52019-06-27 14:08:51 +0100507func (r *rule) In(path ...string) Rule {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800508 r.paths = append(r.paths, cleanPaths(path)...)
509 return r
510}
Colin Crossfd4f7432019-03-05 15:06:16 -0800511
Liz Kammera3d79152021-10-28 18:14:04 -0400512// NotIn adds path(s) to that this rule does not apply to.
Paul Duffin730f2a52019-06-27 14:08:51 +0100513func (r *rule) NotIn(path ...string) Rule {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800514 r.unlessPaths = append(r.unlessPaths, cleanPaths(path)...)
515 return r
516}
Colin Crossfd4f7432019-03-05 15:06:16 -0800517
Liz Kammera3d79152021-10-28 18:14:04 -0400518// InDirectDeps adds dep(s) that are not allowed with this rule.
Paul Duffin35781882019-07-25 15:41:09 +0100519func (r *rule) InDirectDeps(deps ...string) Rule {
520 for _, d := range deps {
521 r.directDeps[d] = true
522 }
523 return r
524}
525
Liz Kammera3d79152021-10-28 18:14:04 -0400526// WithOsClass adds osClass(es) that this rule applies to.
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100527func (r *rule) WithOsClass(osClasses ...OsClass) Rule {
528 r.osClasses = append(r.osClasses, osClasses...)
529 return r
530}
531
Liz Kammera3d79152021-10-28 18:14:04 -0400532// ModuleType adds type(s) that this rule applies to.
Paul Duffin730f2a52019-06-27 14:08:51 +0100533func (r *rule) ModuleType(types ...string) Rule {
Colin Crossfd4f7432019-03-05 15:06:16 -0800534 r.moduleTypes = append(r.moduleTypes, types...)
535 return r
536}
537
Liz Kammera3d79152021-10-28 18:14:04 -0400538// NotModuleType adds type(s) that this rule does not apply to..
Paul Duffin730f2a52019-06-27 14:08:51 +0100539func (r *rule) NotModuleType(types ...string) Rule {
Colin Crossfd4f7432019-03-05 15:06:16 -0800540 r.unlessModuleTypes = append(r.unlessModuleTypes, types...)
541 return r
542}
543
Liz Kammera3d79152021-10-28 18:14:04 -0400544// With specifies property/value combinations that are restricted for this rule.
Paul Duffin730f2a52019-06-27 14:08:51 +0100545func (r *rule) With(properties, value string) Rule {
Paul Duffinc8111702019-07-22 12:13:55 +0100546 return r.WithMatcher(properties, selectMatcher(value))
547}
548
Liz Kammera3d79152021-10-28 18:14:04 -0400549// WithMatcher specifies property/matcher combinations that are restricted for this rule.
Paul Duffinc8111702019-07-22 12:13:55 +0100550func (r *rule) WithMatcher(properties string, matcher ValueMatcher) Rule {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800551 r.props = append(r.props, ruleProperty{
Paul Duffin73bf0542019-07-12 14:12:49 +0100552 fields: fieldNamesForProperties(properties),
Paul Duffinc8111702019-07-22 12:13:55 +0100553 matcher: matcher,
Steven Moreland65b3fd92017-12-06 14:18:35 -0800554 })
555 return r
556}
Colin Crossfd4f7432019-03-05 15:06:16 -0800557
Liz Kammera3d79152021-10-28 18:14:04 -0400558// Without specifies property/value combinations that this rule does not apply to.
Paul Duffin730f2a52019-06-27 14:08:51 +0100559func (r *rule) Without(properties, value string) Rule {
Paul Duffinc8111702019-07-22 12:13:55 +0100560 return r.WithoutMatcher(properties, selectMatcher(value))
561}
562
Liz Kammera3d79152021-10-28 18:14:04 -0400563// Without specifies property/matcher combinations that this rule does not apply to.
Paul Duffinc8111702019-07-22 12:13:55 +0100564func (r *rule) WithoutMatcher(properties string, matcher ValueMatcher) Rule {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800565 r.unlessProps = append(r.unlessProps, ruleProperty{
Paul Duffin73bf0542019-07-12 14:12:49 +0100566 fields: fieldNamesForProperties(properties),
Paul Duffinc8111702019-07-22 12:13:55 +0100567 matcher: matcher,
Steven Moreland65b3fd92017-12-06 14:18:35 -0800568 })
569 return r
570}
Colin Crossfd4f7432019-03-05 15:06:16 -0800571
Paul Duffin73bf0542019-07-12 14:12:49 +0100572func selectMatcher(expected string) ValueMatcher {
573 if expected == "*" {
574 return anyMatcherInstance
575 }
576 return &equalMatcher{expected: expected}
577}
578
Liz Kammera3d79152021-10-28 18:14:04 -0400579// Because specifies a reason for this rule.
Paul Duffin730f2a52019-06-27 14:08:51 +0100580func (r *rule) Because(reason string) Rule {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800581 r.reason = reason
582 return r
583}
584
585func (r *rule) String() string {
Liz Kammera3d79152021-10-28 18:14:04 -0400586 s := []string{"neverallow requirements. Not allowed:"}
587 if len(r.paths) > 0 {
588 s = append(s, fmt.Sprintf("in dirs: %q", r.paths))
Steven Moreland65b3fd92017-12-06 14:18:35 -0800589 }
Liz Kammera3d79152021-10-28 18:14:04 -0400590 if len(r.moduleTypes) > 0 {
591 s = append(s, fmt.Sprintf("module types: %q", r.moduleTypes))
Steven Moreland65b3fd92017-12-06 14:18:35 -0800592 }
Liz Kammera3d79152021-10-28 18:14:04 -0400593 if len(r.props) > 0 {
594 s = append(s, fmt.Sprintf("properties matching: %s", r.props))
Colin Crossfd4f7432019-03-05 15:06:16 -0800595 }
Liz Kammera3d79152021-10-28 18:14:04 -0400596 if len(r.directDeps) > 0 {
Cole Faust18994c72023-02-28 16:02:16 -0800597 s = append(s, fmt.Sprintf("dep(s): %q", SortedKeys(r.directDeps)))
Colin Crossfd4f7432019-03-05 15:06:16 -0800598 }
Liz Kammera3d79152021-10-28 18:14:04 -0400599 if len(r.osClasses) > 0 {
600 s = append(s, fmt.Sprintf("os class(es): %q", r.osClasses))
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100601 }
Liz Kammera3d79152021-10-28 18:14:04 -0400602 if len(r.unlessPaths) > 0 {
603 s = append(s, fmt.Sprintf("EXCEPT in dirs: %q", r.unlessPaths))
604 }
605 if len(r.unlessModuleTypes) > 0 {
606 s = append(s, fmt.Sprintf("EXCEPT module types: %q", r.unlessModuleTypes))
607 }
608 if len(r.unlessProps) > 0 {
609 s = append(s, fmt.Sprintf("EXCEPT properties matching: %q", r.unlessProps))
Andrei Onea115e7e72020-06-05 21:14:03 +0100610 }
Steven Moreland65b3fd92017-12-06 14:18:35 -0800611 if len(r.reason) != 0 {
Liz Kammera3d79152021-10-28 18:14:04 -0400612 s = append(s, " which is restricted because "+r.reason)
Steven Moreland65b3fd92017-12-06 14:18:35 -0800613 }
Liz Kammera3d79152021-10-28 18:14:04 -0400614 if len(s) == 1 {
615 s[0] = "neverallow requirements (empty)"
616 }
617 return strings.Join(s, "\n\t")
Steven Moreland65b3fd92017-12-06 14:18:35 -0800618}
619
620func (r *rule) appliesToPath(dir string) bool {
Jaewoong Jung3aff5782020-02-11 07:54:35 -0800621 includePath := len(r.paths) == 0 || HasAnyPrefix(dir, r.paths)
622 excludePath := HasAnyPrefix(dir, r.unlessPaths)
Steven Moreland65b3fd92017-12-06 14:18:35 -0800623 return includePath && !excludePath
624}
625
Paul Duffin35781882019-07-25 15:41:09 +0100626func (r *rule) appliesToDirectDeps(ctx BottomUpMutatorContext) bool {
627 if len(r.directDeps) == 0 {
628 return true
629 }
630
631 matches := false
632 ctx.VisitDirectDeps(func(m Module) {
633 if !matches {
634 name := ctx.OtherModuleName(m)
635 matches = r.directDeps[name]
636 }
637 })
638
639 return matches
640}
641
Paul Duffinf1c9bbe2019-07-26 10:48:06 +0100642func (r *rule) appliesToOsClass(osClass OsClass) bool {
643 if len(r.osClasses) == 0 {
644 return true
645 }
646
647 for _, c := range r.osClasses {
648 if c == osClass {
649 return true
650 }
651 }
652
653 return false
654}
655
Colin Crossfd4f7432019-03-05 15:06:16 -0800656func (r *rule) appliesToModuleType(moduleType string) bool {
657 return (len(r.moduleTypes) == 0 || InList(moduleType, r.moduleTypes)) && !InList(moduleType, r.unlessModuleTypes)
658}
659
Cole Faust5b35cb92024-08-27 15:47:06 -0700660func (r *rule) appliesToProperties(ctx BottomUpMutatorContext, properties []interface{}) bool {
661 includeProps := hasAllProperties(ctx, properties, r.props)
662 excludeProps := hasAnyProperty(ctx, properties, r.unlessProps)
Steven Moreland65b3fd92017-12-06 14:18:35 -0800663 return includeProps && !excludeProps
664}
665
Paul Duffinc8111702019-07-22 12:13:55 +0100666func StartsWith(prefix string) ValueMatcher {
667 return &startsWithMatcher{prefix}
668}
669
Anton Hansson45376402020-04-09 14:18:21 +0100670func Regexp(re string) ValueMatcher {
671 r, err := regexp.Compile(re)
672 if err != nil {
673 panic(err)
674 }
675 return &regexMatcher{r}
676}
677
Andrei Onea115e7e72020-06-05 21:14:03 +0100678func NotInList(allowed []string) ValueMatcher {
679 return &notInListMatcher{allowed}
680}
681
Steven Moreland65b3fd92017-12-06 14:18:35 -0800682// assorted utils
683
684func cleanPaths(paths []string) []string {
685 res := make([]string, len(paths))
686 for i, v := range paths {
687 res[i] = filepath.Clean(v) + "/"
688 }
689 return res
690}
691
692func fieldNamesForProperties(propertyNames string) []string {
693 names := strings.Split(propertyNames, ".")
694 for i, v := range names {
695 names[i] = proptools.FieldNameForProperty(v)
696 }
697 return names
698}
699
Cole Faust5b35cb92024-08-27 15:47:06 -0700700func hasAnyProperty(ctx BottomUpMutatorContext, properties []interface{}, props []ruleProperty) bool {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800701 for _, v := range props {
Cole Faust5b35cb92024-08-27 15:47:06 -0700702 if hasProperty(ctx, properties, v) {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800703 return true
704 }
705 }
706 return false
707}
708
Cole Faust5b35cb92024-08-27 15:47:06 -0700709func hasAllProperties(ctx BottomUpMutatorContext, properties []interface{}, props []ruleProperty) bool {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800710 for _, v := range props {
Cole Faust5b35cb92024-08-27 15:47:06 -0700711 if !hasProperty(ctx, properties, v) {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800712 return false
713 }
714 }
715 return true
716}
717
Cole Faust5b35cb92024-08-27 15:47:06 -0700718func hasProperty(ctx BottomUpMutatorContext, properties []interface{}, prop ruleProperty) bool {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800719 for _, propertyStruct := range properties {
720 propertiesValue := reflect.ValueOf(propertyStruct).Elem()
721 for _, v := range prop.fields {
722 if !propertiesValue.IsValid() {
723 break
724 }
725 propertiesValue = propertiesValue.FieldByName(v)
726 }
727 if !propertiesValue.IsValid() {
728 continue
729 }
730
Paul Duffin73bf0542019-07-12 14:12:49 +0100731 check := func(value string) bool {
Anton Hanssone1b18362021-12-23 15:05:38 +0000732 return prop.matcher.Test(value)
Steven Moreland65b3fd92017-12-06 14:18:35 -0800733 }
734
Cole Faust5b35cb92024-08-27 15:47:06 -0700735 if matchValue(ctx, propertiesValue, check) {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800736 return true
737 }
738 }
739 return false
740}
741
Cole Faust5b35cb92024-08-27 15:47:06 -0700742func matchValue(ctx BottomUpMutatorContext, value reflect.Value, check func(string) bool) bool {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800743 if !value.IsValid() {
744 return false
745 }
746
747 if value.Kind() == reflect.Ptr {
748 if value.IsNil() {
749 return check("")
750 }
751 value = value.Elem()
752 }
753
Cole Faust5b35cb92024-08-27 15:47:06 -0700754 switch v := value.Interface().(type) {
755 case string:
756 return check(v)
757 case bool:
758 return check(strconv.FormatBool(v))
759 case int:
760 return check(strconv.FormatInt((int64)(v), 10))
761 case []string:
762 for _, v := range v {
763 if check(v) {
764 return true
765 }
Steven Moreland65b3fd92017-12-06 14:18:35 -0800766 }
Cole Faust5b35cb92024-08-27 15:47:06 -0700767 return false
768 case proptools.Configurable[string]:
769 return check(v.GetOrDefault(ctx, ""))
770 case proptools.Configurable[bool]:
771 return check(strconv.FormatBool(v.GetOrDefault(ctx, false)))
772 case proptools.Configurable[[]string]:
773 for _, v := range v.GetOrDefault(ctx, nil) {
Steven Moreland65b3fd92017-12-06 14:18:35 -0800774 if check(v) {
775 return true
776 }
777 }
778 return false
779 }
780
781 panic("Can't handle type: " + value.Kind().String())
782}
Paul Duffin115445b2019-08-07 15:31:07 +0100783
784var neverallowRulesKey = NewOnceKey("neverallowRules")
785
786func neverallowRules(config Config) []Rule {
787 return config.Once(neverallowRulesKey, func() interface{} {
788 // No test rules were set by setTestNeverallowRules, use the global rules
789 return neverallows
790 }).([]Rule)
791}
792
793// Overrides the default neverallow rules for the supplied config.
794//
795// For testing only.
Paul Duffin45338f02021-03-30 23:07:52 +0100796func setTestNeverallowRules(config Config, testRules []Rule) {
Paul Duffin115445b2019-08-07 15:31:07 +0100797 config.Once(neverallowRulesKey, func() interface{} { return testRules })
798}
Paul Duffin45338f02021-03-30 23:07:52 +0100799
800// Prepares for a test by setting neverallow rules and enabling the mutator.
801//
802// If the supplied rules are nil then the default rules are used.
803func PrepareForTestWithNeverallowRules(testRules []Rule) FixturePreparer {
804 return GroupFixturePreparers(
805 FixtureModifyConfig(func(config Config) {
806 if testRules != nil {
807 setTestNeverallowRules(config, testRules)
808 }
809 }),
810 FixtureRegisterWithContext(func(ctx RegistrationContext) {
811 ctx.PostDepsMutators(registerNeverallowMutator)
812 }),
813 )
814}