blob: 8abd9dec56be32be99a70cf8d929e466885bb422 [file] [log] [blame]
Adam Lesinski2ae4a872015-11-02 16:10:55 -08001/*
2 * Copyright (C) 2015 The Android Open Source Project
3 *
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at
7 *
8 * http://www.apache.org/licenses/LICENSE-2.0
9 *
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
15 */
16
Adam Lesinski2ae4a872015-11-02 16:10:55 -080017#include "link/ManifestFixer.h"
Adam Lesinskice5e56e2016-10-21 17:56:45 -070018
19#include <unordered_set>
20
21#include "android-base/logging.h"
22
Adam Lesinskicacb28f2016-10-19 12:18:14 -070023#include "ResourceUtils.h"
Fabien Sanglard2d34e762019-02-21 15:13:29 -080024#include "trace/TraceBuffer.h"
Adam Lesinski2ae4a872015-11-02 16:10:55 -080025#include "util/Util.h"
Adam Lesinskicc5609d2016-04-05 12:41:07 -070026#include "xml/XmlActionExecutor.h"
Adam Lesinski467f1712015-11-16 17:35:44 -080027#include "xml/XmlDom.h"
Adam Lesinski2ae4a872015-11-02 16:10:55 -080028
Adam Lesinskid5083f62017-01-16 15:07:21 -080029using android::StringPiece;
30
Adam Lesinski2ae4a872015-11-02 16:10:55 -080031namespace aapt {
32
Adam Lesinskifca5e422017-12-20 15:03:36 -080033static bool RequiredNameIsNotEmpty(xml::Element* el, SourcePathDiagnostics* diag) {
34 xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name");
35 if (attr == nullptr) {
36 diag->Error(DiagMessage(el->line_number)
37 << "<" << el->name << "> is missing attribute 'android:name'");
38 return false;
39 }
40
41 if (attr->value.empty()) {
42 diag->Error(DiagMessage(el->line_number)
43 << "attribute 'android:name' in <" << el->name << "> tag must not be empty");
44 return false;
45 }
46 return true;
47}
48
Adam Lesinskib0c47ef2017-03-06 20:05:57 -080049// This is how PackageManager builds class names from AndroidManifest.xml entries.
Adam Lesinskice5e56e2016-10-21 17:56:45 -070050static bool NameIsJavaClassName(xml::Element* el, xml::Attribute* attr,
Adam Lesinskicc5609d2016-04-05 12:41:07 -070051 SourcePathDiagnostics* diag) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -070052 // We allow unqualified class names (ie: .HelloActivity)
53 // Since we don't know the package name, we can just make a fake one here and
54 // the test will be identical as long as the real package name is valid too.
Adam Lesinskice5e56e2016-10-21 17:56:45 -070055 Maybe<std::string> fully_qualified_class_name =
56 util::GetFullyQualifiedClassName("a", attr->value);
Adam Lesinskicc5609d2016-04-05 12:41:07 -070057
Adam Lesinskice5e56e2016-10-21 17:56:45 -070058 StringPiece qualified_class_name = fully_qualified_class_name
59 ? fully_qualified_class_name.value()
60 : attr->value;
Adam Lesinskid0f116b2016-07-08 15:00:32 -070061
Adam Lesinskice5e56e2016-10-21 17:56:45 -070062 if (!util::IsJavaClassName(qualified_class_name)) {
63 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070064 << "attribute 'android:name' in <" << el->name
65 << "> tag must be a valid Java class name");
Adam Lesinski52364f72016-01-11 13:10:24 -080066 return false;
Adam Lesinskicacb28f2016-10-19 12:18:14 -070067 }
68 return true;
69}
70
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080071static bool OptionalNameIsJavaClassName(xml::Element* el, SourcePathDiagnostics* diag) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -070072 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
73 return NameIsJavaClassName(el, attr, diag);
Adam Lesinskicacb28f2016-10-19 12:18:14 -070074 }
75 return true;
76}
77
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080078static bool RequiredNameIsJavaClassName(xml::Element* el, SourcePathDiagnostics* diag) {
Adam Lesinskifca5e422017-12-20 15:03:36 -080079 xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name");
80 if (attr == nullptr) {
81 diag->Error(DiagMessage(el->line_number)
82 << "<" << el->name << "> is missing attribute 'android:name'");
83 return false;
Adam Lesinskicacb28f2016-10-19 12:18:14 -070084 }
Adam Lesinskifca5e422017-12-20 15:03:36 -080085 return NameIsJavaClassName(el, attr, diag);
Adam Lesinski52364f72016-01-11 13:10:24 -080086}
87
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080088static bool RequiredNameIsJavaPackage(xml::Element* el, SourcePathDiagnostics* diag) {
Adam Lesinskifca5e422017-12-20 15:03:36 -080089 xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name");
90 if (attr == nullptr) {
91 diag->Error(DiagMessage(el->line_number)
92 << "<" << el->name << "> is missing attribute 'android:name'");
93 return false;
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080094 }
Adam Lesinskifca5e422017-12-20 15:03:36 -080095
96 if (!util::IsJavaPackageName(attr->value)) {
97 diag->Error(DiagMessage(el->line_number) << "attribute 'android:name' in <" << el->name
98 << "> tag must be a valid Java package name");
99 return false;
100 }
101 return true;
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -0800102}
103
Adam Lesinskib5dc4bd2017-02-22 19:29:29 -0800104static xml::XmlNodeAction::ActionFuncWithDiag RequiredAndroidAttribute(const std::string& attr) {
105 return [=](xml::Element* el, SourcePathDiagnostics* diag) -> bool {
106 if (el->FindAttribute(xml::kSchemaAndroid, attr) == nullptr) {
107 diag->Error(DiagMessage(el->line_number)
108 << "<" << el->name << "> is missing required attribute 'android:" << attr << "'");
109 return false;
110 }
111 return true;
112 };
113}
114
Todd Kennedyce3e1292020-10-29 17:14:24 -0700115static xml::XmlNodeAction::ActionFuncWithDiag RequiredOneAndroidAttribute(
116 const std::string& attrName1, const std::string& attrName2) {
117 return [=](xml::Element* el, SourcePathDiagnostics* diag) -> bool {
118 xml::Attribute* attr1 = el->FindAttribute(xml::kSchemaAndroid, attrName1);
119 xml::Attribute* attr2 = el->FindAttribute(xml::kSchemaAndroid, attrName2);
120 if (attr1 == nullptr && attr2 == nullptr) {
121 diag->Error(DiagMessage(el->line_number)
122 << "<" << el->name << "> is missing required attribute 'android:" << attrName1
123 << "' or 'android:" << attrName2 << "'");
124 return false;
125 }
126 if (attr1 != nullptr && attr2 != nullptr) {
127 diag->Error(DiagMessage(el->line_number)
128 << "<" << el->name << "> can only specify one of attribute 'android:" << attrName1
129 << "' or 'android:" << attrName2 << "'");
130 return false;
131 }
132 return true;
133 };
134}
135
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800136static bool AutoGenerateIsFeatureSplit(xml::Element* el, SourcePathDiagnostics* diag) {
137 constexpr const char* kFeatureSplit = "featureSplit";
138 constexpr const char* kIsFeatureSplit = "isFeatureSplit";
139
140 xml::Attribute* attr = el->FindAttribute({}, kFeatureSplit);
141 if (attr != nullptr) {
142 // Rewrite the featureSplit attribute to be "split". This is what the
143 // platform recognizes.
144 attr->name = "split";
145
146 // Now inject the android:isFeatureSplit="true" attribute.
147 xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, kIsFeatureSplit);
148 if (attr != nullptr) {
149 if (!ResourceUtils::ParseBool(attr->value).value_or_default(false)) {
150 // The isFeatureSplit attribute is false, which conflicts with the use
151 // of "featureSplit".
152 diag->Error(DiagMessage(el->line_number)
153 << "attribute 'featureSplit' used in <manifest> but 'android:isFeatureSplit' "
154 "is not 'true'");
155 return false;
156 }
157
158 // The attribute is already there and set to true, nothing to do.
159 } else {
160 el->attributes.push_back(xml::Attribute{xml::kSchemaAndroid, kIsFeatureSplit, "true"});
161 }
162 }
163 return true;
164}
165
Rhed Jao2c434422020-11-12 10:48:03 +0800166static bool VerifyManifest(xml::Element* el, xml::XmlActionExecutorPolicy policy,
167 SourcePathDiagnostics* diag) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700168 xml::Attribute* attr = el->FindAttribute({}, "package");
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700169 if (!attr) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700170 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700171 << "<manifest> tag is missing 'package' attribute");
172 return false;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700173 } else if (ResourceUtils::IsReference(attr->value)) {
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800174 diag->Error(DiagMessage(el->line_number)
175 << "attribute 'package' in <manifest> tag must not be a reference");
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700176 return false;
Adam Lesinski96ea08f2017-11-06 10:44:46 -0800177 } else if (!util::IsAndroidPackageName(attr->value)) {
Rhed Jao2c434422020-11-12 10:48:03 +0800178 DiagMessage error_msg(el->line_number);
179 error_msg << "attribute 'package' in <manifest> tag is not a valid Android package name: '"
180 << attr->value << "'";
181 if (policy == xml::XmlActionExecutorPolicy::kAllowListWarning) {
182 // Treat the error only as a warning.
183 diag->Warn(error_msg);
184 } else {
185 diag->Error(error_msg);
186 return false;
187 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700188 }
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800189
190 attr = el->FindAttribute({}, "split");
191 if (attr) {
192 if (!util::IsJavaPackageName(attr->value)) {
193 diag->Error(DiagMessage(el->line_number) << "attribute 'split' in <manifest> tag is not a "
194 "valid split name");
195 return false;
196 }
197 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700198 return true;
Adam Lesinski52364f72016-01-11 13:10:24 -0800199}
200
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800201// The coreApp attribute in <manifest> is not a regular AAPT attribute, so type
202// checking on it is manual.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700203static bool FixCoreAppAttribute(xml::Element* el, SourcePathDiagnostics* diag) {
204 if (xml::Attribute* attr = el->FindAttribute("", "coreApp")) {
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800205 std::unique_ptr<BinaryPrimitive> result = ResourceUtils::TryParseBool(attr->value);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700206 if (!result) {
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800207 diag->Error(DiagMessage(el->line_number) << "attribute coreApp must be a boolean");
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700208 return false;
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700209 }
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700210 attr->compiled_value = std::move(result);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700211 }
212 return true;
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700213}
214
Adam Lesinski86d67df2017-01-31 13:47:27 -0800215// Checks that <uses-feature> has android:glEsVersion or android:name, not both (or neither).
216static bool VerifyUsesFeature(xml::Element* el, SourcePathDiagnostics* diag) {
217 bool has_name = false;
218 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
219 if (attr->value.empty()) {
220 diag->Error(DiagMessage(el->line_number)
221 << "android:name in <uses-feature> must not be empty");
222 return false;
223 }
224 has_name = true;
225 }
226
227 bool has_gl_es_version = false;
228 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "glEsVersion")) {
229 if (has_name) {
230 diag->Error(DiagMessage(el->line_number)
231 << "cannot define both android:name and android:glEsVersion in <uses-feature>");
232 return false;
233 }
234 has_gl_es_version = true;
235 }
236
237 if (!has_name && !has_gl_es_version) {
238 diag->Error(DiagMessage(el->line_number)
239 << "<uses-feature> must have either android:name or android:glEsVersion attribute");
240 return false;
241 }
242 return true;
243}
244
Donald Chai6e497352019-05-19 21:07:50 -0700245// Ensure that 'ns_decls' contains a declaration for 'uri', using 'prefix' as
246// the xmlns prefix if possible.
247static void EnsureNamespaceIsDeclared(const std::string& prefix, const std::string& uri,
248 std::vector<xml::NamespaceDecl>* ns_decls) {
249 if (std::find_if(ns_decls->begin(), ns_decls->end(), [&](const xml::NamespaceDecl& ns_decl) {
250 return ns_decl.uri == uri;
251 }) != ns_decls->end()) {
252 return;
253 }
254
255 std::set<std::string> used_prefixes;
256 for (const auto& ns_decl : *ns_decls) {
257 used_prefixes.insert(ns_decl.prefix);
258 }
259
260 // Make multiple attempts in the unlikely event that 'prefix' is already taken.
261 std::string disambiguator;
262 for (int i = 0; i < used_prefixes.size() + 1; i++) {
263 std::string attempted_prefix = prefix + disambiguator;
264 if (used_prefixes.find(attempted_prefix) == used_prefixes.end()) {
265 ns_decls->push_back(xml::NamespaceDecl{attempted_prefix, uri});
266 return;
267 }
268 disambiguator = std::to_string(i);
269 }
270}
271
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700272bool ManifestFixer::BuildRules(xml::XmlActionExecutor* executor,
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700273 IDiagnostics* diag) {
274 // First verify some options.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700275 if (options_.rename_manifest_package) {
276 if (!util::IsJavaPackageName(options_.rename_manifest_package.value())) {
277 diag->Error(DiagMessage() << "invalid manifest package override '"
278 << options_.rename_manifest_package.value()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700279 << "'");
280 return false;
281 }
282 }
283
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700284 if (options_.rename_instrumentation_target_package) {
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800285 if (!util::IsJavaPackageName(options_.rename_instrumentation_target_package.value())) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700286 diag->Error(DiagMessage()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700287 << "invalid instrumentation target package override '"
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700288 << options_.rename_instrumentation_target_package.value()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700289 << "'");
290 return false;
291 }
292 }
293
Roshan Piusae12ce42020-04-25 16:08:55 -0700294 if (options_.rename_overlay_target_package) {
295 if (!util::IsJavaPackageName(options_.rename_overlay_target_package.value())) {
296 diag->Error(DiagMessage()
297 << "invalid overlay target package override '"
298 << options_.rename_overlay_target_package.value()
299 << "'");
300 return false;
301 }
302 }
303
Adam Lesinski86d67df2017-01-31 13:47:27 -0800304 // Common <intent-filter> actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700305 xml::XmlNodeAction intent_filter_action;
Adam Lesinskifca5e422017-12-20 15:03:36 -0800306 intent_filter_action["action"].Action(RequiredNameIsNotEmpty);
307 intent_filter_action["category"].Action(RequiredNameIsNotEmpty);
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700308 intent_filter_action["data"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700309
Adam Lesinski86d67df2017-01-31 13:47:27 -0800310 // Common <meta-data> actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700311 xml::XmlNodeAction meta_data_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700312
Todd Kennedyce3e1292020-10-29 17:14:24 -0700313 // Common <property> actions.
314 xml::XmlNodeAction property_action;
315 property_action.Action(RequiredOneAndroidAttribute("resource", "value"));
316
Adam Lesinski86d67df2017-01-31 13:47:27 -0800317 // Common <uses-feature> actions.
318 xml::XmlNodeAction uses_feature_action;
319 uses_feature_action.Action(VerifyUsesFeature);
320
321 // Common component actions.
322 xml::XmlNodeAction component_action;
323 component_action.Action(RequiredNameIsJavaClassName);
324 component_action["intent-filter"] = intent_filter_action;
Ryan Mitchell28afe682018-09-07 14:33:14 -0700325 component_action["preferred"] = intent_filter_action;
Adam Lesinski86d67df2017-01-31 13:47:27 -0800326 component_action["meta-data"] = meta_data_action;
Todd Kennedyce3e1292020-10-29 17:14:24 -0700327 component_action["property"] = property_action;
Adam Lesinski86d67df2017-01-31 13:47:27 -0800328
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700329 // Manifest actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700330 xml::XmlNodeAction& manifest_action = (*executor)["manifest"];
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800331 manifest_action.Action(AutoGenerateIsFeatureSplit);
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700332 manifest_action.Action(VerifyManifest);
333 manifest_action.Action(FixCoreAppAttribute);
334 manifest_action.Action([&](xml::Element* el) -> bool {
Donald Chai6e497352019-05-19 21:07:50 -0700335 EnsureNamespaceIsDeclared("android", xml::kSchemaAndroid, &el->namespace_decls);
336
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700337 if (options_.version_name_default) {
Colin Crossdcd58c42018-05-25 22:46:35 -0700338 if (options_.replace_version) {
339 el->RemoveAttribute(xml::kSchemaAndroid, "versionName");
340 }
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700341 if (el->FindAttribute(xml::kSchemaAndroid, "versionName") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700342 el->attributes.push_back(
343 xml::Attribute{xml::kSchemaAndroid, "versionName",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700344 options_.version_name_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700345 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700346 }
347
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700348 if (options_.version_code_default) {
Colin Crossdcd58c42018-05-25 22:46:35 -0700349 if (options_.replace_version) {
350 el->RemoveAttribute(xml::kSchemaAndroid, "versionCode");
351 }
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700352 if (el->FindAttribute(xml::kSchemaAndroid, "versionCode") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700353 el->attributes.push_back(
354 xml::Attribute{xml::kSchemaAndroid, "versionCode",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700355 options_.version_code_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700356 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700357 }
Ryan Mitchell7cb82a82018-05-10 15:35:31 -0700358
Ryan Mitchell704090e2018-07-31 14:59:25 -0700359 if (options_.version_code_major_default) {
360 if (options_.replace_version) {
361 el->RemoveAttribute(xml::kSchemaAndroid, "versionCodeMajor");
362 }
363 if (el->FindAttribute(xml::kSchemaAndroid, "versionCodeMajor") == nullptr) {
364 el->attributes.push_back(
365 xml::Attribute{xml::kSchemaAndroid, "versionCodeMajor",
366 options_.version_code_major_default.value()});
367 }
368 }
369
Rhed Jaob9ccb8a42020-11-30 21:42:16 +0800370 if (options_.revision_code_default) {
371 if (options_.replace_version) {
372 el->RemoveAttribute(xml::kSchemaAndroid, "revisionCode");
373 }
374 if (el->FindAttribute(xml::kSchemaAndroid, "revisionCode") == nullptr) {
375 el->attributes.push_back(xml::Attribute{xml::kSchemaAndroid, "revisionCode",
376 options_.revision_code_default.value()});
377 }
378 }
379
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800380 return true;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700381 });
382
383 // Meta tags.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700384 manifest_action["eat-comment"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700385
386 // Uses-sdk actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700387 manifest_action["uses-sdk"].Action([&](xml::Element* el) -> bool {
388 if (options_.min_sdk_version_default &&
389 el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700390 // There was no minSdkVersion defined and we have a default to assign.
391 el->attributes.push_back(
392 xml::Attribute{xml::kSchemaAndroid, "minSdkVersion",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700393 options_.min_sdk_version_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700394 }
395
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700396 if (options_.target_sdk_version_default &&
397 el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700398 // There was no targetSdkVersion defined and we have a default to assign.
399 el->attributes.push_back(
400 xml::Attribute{xml::kSchemaAndroid, "targetSdkVersion",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700401 options_.target_sdk_version_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700402 }
403 return true;
404 });
Anton Hanssonb2f709d2020-01-09 10:25:23 +0000405 manifest_action["uses-sdk"]["extension-sdk"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700406
407 // Instrumentation actions.
Adam Lesinski86d67df2017-01-31 13:47:27 -0800408 manifest_action["instrumentation"].Action(RequiredNameIsJavaClassName);
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700409 manifest_action["instrumentation"].Action([&](xml::Element* el) -> bool {
410 if (!options_.rename_instrumentation_target_package) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700411 return true;
412 }
413
414 if (xml::Attribute* attr =
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700415 el->FindAttribute(xml::kSchemaAndroid, "targetPackage")) {
416 attr->value = options_.rename_instrumentation_target_package.value();
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700417 }
418 return true;
419 });
Adam Lesinski86d67df2017-01-31 13:47:27 -0800420 manifest_action["instrumentation"]["meta-data"] = meta_data_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700421
Philip P. Moltmann12ac3f42020-03-05 15:01:29 -0800422 manifest_action["attribution"];
423 manifest_action["attribution"]["inherit-from"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700424 manifest_action["original-package"];
Roshan Piusae12ce42020-04-25 16:08:55 -0700425 manifest_action["overlay"].Action([&](xml::Element* el) -> bool {
426 if (!options_.rename_overlay_target_package) {
427 return true;
428 }
429
430 if (xml::Attribute* attr =
431 el->FindAttribute(xml::kSchemaAndroid, "targetPackage")) {
432 attr->value = options_.rename_overlay_target_package.value();
433 }
434 return true;
435 });
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700436 manifest_action["protected-broadcast"];
Alan Viverettecf5326f2018-01-05 16:03:50 -0500437 manifest_action["adopt-permissions"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700438 manifest_action["uses-permission"];
Sergey Nikolaienkov65f90992020-09-30 08:17:09 +0000439 manifest_action["uses-permission"]["required-feature"].Action(RequiredNameIsNotEmpty);
440 manifest_action["uses-permission"]["required-not-feature"].Action(RequiredNameIsNotEmpty);
Adam Lesinski4b585db2017-05-12 15:25:50 -0700441 manifest_action["uses-permission-sdk-23"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700442 manifest_action["permission"];
Ryan Mitchell66f6cfb2018-07-25 16:15:17 -0700443 manifest_action["permission"]["meta-data"] = meta_data_action;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700444 manifest_action["permission-tree"];
445 manifest_action["permission-group"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700446 manifest_action["uses-configuration"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700447 manifest_action["supports-screens"];
Adam Lesinski86d67df2017-01-31 13:47:27 -0800448 manifest_action["uses-feature"] = uses_feature_action;
449 manifest_action["feature-group"]["uses-feature"] = uses_feature_action;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700450 manifest_action["compatible-screens"];
451 manifest_action["compatible-screens"]["screen"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700452 manifest_action["supports-gl-texture"];
Ryan Mitchell66f6cfb2018-07-25 16:15:17 -0700453 manifest_action["restrict-update"];
454 manifest_action["package-verifier"];
Adam Lesinski5119e512016-12-05 19:48:20 -0800455 manifest_action["meta-data"] = meta_data_action;
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800456 manifest_action["uses-split"].Action(RequiredNameIsJavaPackage);
Patrick Baumanna4ffb452019-06-24 15:01:49 -0700457 manifest_action["queries"]["package"].Action(RequiredNameIsJavaPackage);
458 manifest_action["queries"]["intent"] = intent_filter_action;
Patrick Baumann99181232020-01-28 10:55:25 -0800459 manifest_action["queries"]["provider"].Action(RequiredAndroidAttribute("authorities"));
Patrick Baumanna4ffb452019-06-24 15:01:49 -0700460 // TODO: more complicated component name tag
Adam Lesinski5119e512016-12-05 19:48:20 -0800461
Adam Lesinski87f1e0f2017-06-27 16:21:58 -0700462 manifest_action["key-sets"]["key-set"]["public-key"];
463 manifest_action["key-sets"]["upgrade-key-set"];
464
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700465 // Application actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700466 xml::XmlNodeAction& application_action = manifest_action["application"];
467 application_action.Action(OptionalNameIsJavaClassName);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700468
Adam Lesinskifca5e422017-12-20 15:03:36 -0800469 application_action["uses-library"].Action(RequiredNameIsNotEmpty);
Jiyong Park6a5b8b12020-06-30 13:23:36 +0900470 application_action["uses-native-library"].Action(RequiredNameIsNotEmpty);
Adam Lesinskifca5e422017-12-20 15:03:36 -0800471 application_action["library"].Action(RequiredNameIsNotEmpty);
Chris Craik335b5652019-04-04 12:46:47 -0700472 application_action["profileable"];
Todd Kennedyce3e1292020-10-29 17:14:24 -0700473 application_action["property"] = property_action;
Adam Lesinskib5dc4bd2017-02-22 19:29:29 -0800474
475 xml::XmlNodeAction& static_library_action = application_action["static-library"];
476 static_library_action.Action(RequiredNameIsJavaPackage);
477 static_library_action.Action(RequiredAndroidAttribute("version"));
478
479 xml::XmlNodeAction& uses_static_library_action = application_action["uses-static-library"];
480 uses_static_library_action.Action(RequiredNameIsJavaPackage);
481 uses_static_library_action.Action(RequiredAndroidAttribute("version"));
482 uses_static_library_action.Action(RequiredAndroidAttribute("certDigest"));
Ryan Mitchell66f6cfb2018-07-25 16:15:17 -0700483 uses_static_library_action["additional-certificate"];
Adam Lesinskib5dc4bd2017-02-22 19:29:29 -0800484
Dianne Hackborn813d7502018-10-02 16:59:46 -0700485 xml::XmlNodeAction& uses_package_action = application_action["uses-package"];
486 uses_package_action.Action(RequiredNameIsJavaPackage);
487 uses_package_action["additional-certificate"];
488
Ryan Mitchelle5b38a62018-03-23 13:35:00 -0700489 if (options_.debug_mode) {
490 application_action.Action([&](xml::Element* el) -> bool {
491 xml::Attribute *attr = el->FindOrCreateAttribute(xml::kSchemaAndroid, "debuggable");
492 attr->value = "true";
493 return true;
494 });
495 }
496
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700497 application_action["meta-data"] = meta_data_action;
Adam Lesinski7a917a22017-06-02 12:55:24 -0700498
Dianne Hackbornfc0839a2020-01-31 11:02:28 -0800499 application_action["processes"];
500 application_action["processes"]["deny-permission"];
501 application_action["processes"]["allow-permission"];
502 application_action["processes"]["process"]["deny-permission"];
503 application_action["processes"]["process"]["allow-permission"];
504
Adam Lesinski86d67df2017-01-31 13:47:27 -0800505 application_action["activity"] = component_action;
Adam Lesinski7a917a22017-06-02 12:55:24 -0700506 application_action["activity"]["layout"];
507
Adam Lesinski86d67df2017-01-31 13:47:27 -0800508 application_action["activity-alias"] = component_action;
509 application_action["service"] = component_action;
510 application_action["receiver"] = component_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700511
512 // Provider actions.
Adam Lesinski86d67df2017-01-31 13:47:27 -0800513 application_action["provider"] = component_action;
Adam Lesinskic10c0d02017-04-28 12:54:08 -0700514 application_action["provider"]["grant-uri-permission"];
Adam Lesinski25783ca2017-04-24 13:33:47 -0700515 application_action["provider"]["path-permission"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700516
Ryan Mitchell28afe682018-09-07 14:33:14 -0700517 manifest_action["package"] = manifest_action;
518
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700519 return true;
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800520}
521
Adam Lesinski23034b92017-11-29 16:27:44 -0800522static void FullyQualifyClassName(const StringPiece& package, const StringPiece& attr_ns,
523 const StringPiece& attr_name, xml::Element* el) {
524 xml::Attribute* attr = el->FindAttribute(attr_ns, attr_name);
525 if (attr != nullptr) {
526 if (Maybe<std::string> new_value = util::GetFullyQualifiedClassName(package, attr->value)) {
527 attr->value = std::move(new_value.value());
Adam Lesinski52364f72016-01-11 13:10:24 -0800528 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700529 }
Adam Lesinski23034b92017-11-29 16:27:44 -0800530}
Adam Lesinski52364f72016-01-11 13:10:24 -0800531
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800532static bool RenameManifestPackage(const StringPiece& package_override, xml::Element* manifest_el) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700533 xml::Attribute* attr = manifest_el->FindAttribute({}, "package");
Adam Lesinski52364f72016-01-11 13:10:24 -0800534
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700535 // We've already verified that the manifest element is present, with a package
536 // name specified.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700537 CHECK(attr != nullptr);
Adam Lesinski52364f72016-01-11 13:10:24 -0800538
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700539 std::string original_package = std::move(attr->value);
Adam Lesinskid5083f62017-01-16 15:07:21 -0800540 attr->value = package_override.to_string();
Adam Lesinski52364f72016-01-11 13:10:24 -0800541
Adam Lesinski23034b92017-11-29 16:27:44 -0800542 xml::Element* application_el = manifest_el->FindChild({}, "application");
543 if (application_el != nullptr) {
544 FullyQualifyClassName(original_package, xml::kSchemaAndroid, "name", application_el);
545 FullyQualifyClassName(original_package, xml::kSchemaAndroid, "backupAgent", application_el);
546
547 for (xml::Element* child_el : application_el->GetChildElements()) {
548 if (child_el->namespace_uri.empty()) {
549 if (child_el->name == "activity" || child_el->name == "activity-alias" ||
550 child_el->name == "provider" || child_el->name == "receiver" ||
551 child_el->name == "service") {
552 FullyQualifyClassName(original_package, xml::kSchemaAndroid, "name", child_el);
553 }
554
555 if (child_el->name == "activity-alias") {
556 FullyQualifyClassName(original_package, xml::kSchemaAndroid, "targetActivity", child_el);
557 }
558 }
559 }
560 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700561 return true;
Adam Lesinski52364f72016-01-11 13:10:24 -0800562}
563
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700564bool ManifestFixer::Consume(IAaptContext* context, xml::XmlResource* doc) {
Fabien Sanglard2d34e762019-02-21 15:13:29 -0800565 TRACE_CALL();
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700566 xml::Element* root = xml::FindRootElement(doc->root.get());
567 if (!root || !root->namespace_uri.empty() || root->name != "manifest") {
568 context->GetDiagnostics()->Error(DiagMessage(doc->file.source)
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700569 << "root tag must be <manifest>");
570 return false;
571 }
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800572
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800573 if ((options_.min_sdk_version_default || options_.target_sdk_version_default) &&
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700574 root->FindChild({}, "uses-sdk") == nullptr) {
Adam Lesinskie343eb12016-10-27 16:31:58 -0700575 // Auto insert a <uses-sdk> element. This must be inserted before the
576 // <application> tag. The device runtime PackageParser will make SDK version
577 // decisions while parsing <application>.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700578 std::unique_ptr<xml::Element> uses_sdk = util::make_unique<xml::Element>();
579 uses_sdk->name = "uses-sdk";
Adam Lesinskie343eb12016-10-27 16:31:58 -0700580 root->InsertChild(0, std::move(uses_sdk));
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700581 }
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800582
Adam Lesinskic6284372017-12-04 13:46:23 -0800583 if (options_.compile_sdk_version) {
584 xml::Attribute* attr = root->FindOrCreateAttribute(xml::kSchemaAndroid, "compileSdkVersion");
585
586 // Make sure we un-compile the value if it was set to something else.
587 attr->compiled_value = {};
Adam Lesinskic6284372017-12-04 13:46:23 -0800588 attr->value = options_.compile_sdk_version.value();
Ryan Mitchellaada89c2019-02-12 08:06:26 -0800589
590 attr = root->FindOrCreateAttribute("", "platformBuildVersionCode");
591
592 // Make sure we un-compile the value if it was set to something else.
593 attr->compiled_value = {};
594 attr->value = options_.compile_sdk_version.value();
595
Adam Lesinskic6284372017-12-04 13:46:23 -0800596 }
597
598 if (options_.compile_sdk_version_codename) {
599 xml::Attribute* attr =
600 root->FindOrCreateAttribute(xml::kSchemaAndroid, "compileSdkVersionCodename");
601
602 // Make sure we un-compile the value if it was set to something else.
603 attr->compiled_value = {};
Ryan Mitchellaada89c2019-02-12 08:06:26 -0800604 attr->value = options_.compile_sdk_version_codename.value();
Adam Lesinskic6284372017-12-04 13:46:23 -0800605
Ryan Mitchellaada89c2019-02-12 08:06:26 -0800606 attr = root->FindOrCreateAttribute("", "platformBuildVersionName");
607
608 // Make sure we un-compile the value if it was set to something else.
609 attr->compiled_value = {};
Adam Lesinskic6284372017-12-04 13:46:23 -0800610 attr->value = options_.compile_sdk_version_codename.value();
611 }
612
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700613 xml::XmlActionExecutor executor;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700614 if (!BuildRules(&executor, context->GetDiagnostics())) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700615 return false;
616 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700617
Izabela Orlowskaad9e1322017-12-19 16:22:42 +0000618 xml::XmlActionExecutorPolicy policy = options_.warn_validation
Ryan Mitchell4ea90752020-07-31 08:21:43 -0700619 ? xml::XmlActionExecutorPolicy::kAllowListWarning
620 : xml::XmlActionExecutorPolicy::kAllowList;
Izabela Orlowskaad9e1322017-12-19 16:22:42 +0000621 if (!executor.Execute(policy, context->GetDiagnostics(), doc)) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700622 return false;
623 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700624
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700625 if (options_.rename_manifest_package) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700626 // Rename manifest package outside of the XmlActionExecutor.
Adam Lesinskie343eb12016-10-27 16:31:58 -0700627 // We need to extract the old package name and FullyQualify all class
628 // names.
Adam Lesinskib0c47ef2017-03-06 20:05:57 -0800629 if (!RenameManifestPackage(options_.rename_manifest_package.value(), root)) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700630 return false;
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700631 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700632 }
633 return true;
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800634}
635
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700636} // namespace aapt