refactor: split out string block vs res table string (#3267)

This commit is contained in:
Connor Tumbleson 2023-08-13 13:46:47 -04:00 committed by GitHub
parent 1243dd5546
commit fb6cd883c0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -407,28 +407,13 @@ public class AXmlResourceParser implements XmlResourceParser {
String stringBlockValue = valueRaw == -1 ? null : ResXmlEncoders.escapeXmlChars(mStringBlock.getString(valueRaw)); String stringBlockValue = valueRaw == -1 ? null : ResXmlEncoders.escapeXmlChars(mStringBlock.getString(valueRaw));
String resourceMapValue = null; String resourceMapValue = null;
// Ensure we only track down obfuscated values for reference/attribute type values. Otherwise we might // Ensure we only track down obfuscated values for reference/attribute type values. Otherwise, we might
// spam lookups against resource table for invalid ids. // spam lookups against resource table for invalid ids.
if (valueType == TypedValue.TYPE_REFERENCE || valueType == TypedValue.TYPE_DYNAMIC_REFERENCE || if (valueType == TypedValue.TYPE_REFERENCE || valueType == TypedValue.TYPE_DYNAMIC_REFERENCE ||
valueType == TypedValue.TYPE_ATTRIBUTE || valueType == TypedValue.TYPE_DYNAMIC_ATTRIBUTE) { valueType == TypedValue.TYPE_ATTRIBUTE || valueType == TypedValue.TYPE_DYNAMIC_ATTRIBUTE) {
resourceMapValue = decodeFromResourceId(valueData); resourceMapValue = decodeFromResourceId(valueData);
} }
String value = stringBlockValue; String value = getPreferredString(stringBlockValue, resourceMapValue);
if (stringBlockValue != null && resourceMapValue != null) {
int slashPos = stringBlockValue.lastIndexOf("/");
int colonPos = stringBlockValue.lastIndexOf(":");
// Handle a value with a format of "@yyy/xxx", but avoid "@yyy/zzz:xxx"
if (slashPos != -1) {
if (colonPos == -1) {
String type = stringBlockValue.substring(0, slashPos);
value = type + "/" + resourceMapValue;
}
} else if (! stringBlockValue.equals(resourceMapValue)) {
value = resourceMapValue;
}
}
// try to decode from resource table // try to decode from resource table
int attrResId = getAttributeNameResource(index); int attrResId = getAttributeNameResource(index);
@ -662,6 +647,26 @@ public class AXmlResourceParser implements XmlResourceParser {
return -1; return -1;
} }
private static String getPreferredString(String stringBlockValue, String resourceMapValue) {
String value = stringBlockValue;
if (stringBlockValue != null && resourceMapValue != null) {
int slashPos = stringBlockValue.lastIndexOf("/");
int colonPos = stringBlockValue.lastIndexOf(":");
// Handle a value with a format of "@yyy/xxx", but avoid "@yyy/zzz:xxx"
if (slashPos != -1) {
if (colonPos == -1) {
String type = stringBlockValue.substring(0, slashPos);
value = type + "/" + resourceMapValue;
}
} else if (! stringBlockValue.equals(resourceMapValue)) {
value = resourceMapValue;
}
}
return value;
}
private void resetEventInfo() { private void resetEventInfo() {
mEvent = -1; mEvent = -1;
mLineNumber = -1; mLineNumber = -1;