Update InstrumentedType.java to check instrumented classfile is in valid Unicode namespace instead (#1613)

leerjae · web-flow · commit 95b083f42e5e · 2024-04-04T23:51:13.000+02:00
diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/dynamic/scaffold/InstrumentedType.java b/byte-buddy-dep/src/main/java/net/bytebuddy/dynamic/scaffold/InstrumentedType.java
@@ -1842,13 +1842,13 @@ private static boolean isValidIdentifier(String[] identifier) {
          * @return {@code true} if the given identifier is valid.
          */
         private static boolean isValidIdentifier(String identifier) {
-            if (KEYWORDS.contains(identifier) || identifier.length() == 0 || !Character.isJavaIdentifierStart(identifier.charAt(0))) {
+            if (KEYWORDS.contains(identifier) || identifier.length() == 0 || !Character.isUnicodeIdentifierStart(identifier.charAt(0))) {
                 return false;
             } else if (identifier.equals(PackageDescription.PACKAGE_CLASS_NAME)) {
                 return true;
             }
             for (int index = 1; index < identifier.length(); index++) {
-                if (!Character.isJavaIdentifierPart(identifier.charAt(index))) {
+                if (!Character.isUnicodeIdentifierPart(identifier.charAt(index))) {
                     return false;
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -1842,13 +1842,13 @@ private static boolean isValidIdentifier(String[] identifier) {`
`1842`	`1842`	`* @return {@code true} if the given identifier is valid.`
`1843`	`1843`	`*/`
`1844`	`1844`	`private static boolean isValidIdentifier(String identifier) {`
`1845`		`- if (KEYWORDS.contains(identifier) \|\| identifier.length() == 0 \|\| !Character.isJavaIdentifierStart(identifier.charAt(0))) {`
	`1845`	`+ if (KEYWORDS.contains(identifier) \|\| identifier.length() == 0 \|\| !Character.isUnicodeIdentifierStart(identifier.charAt(0))) {`
`1846`	`1846`	`return false;`
`1847`	`1847`	`} else if (identifier.equals(PackageDescription.PACKAGE_CLASS_NAME)) {`
`1848`	`1848`	`return true;`
`1849`	`1849`	`}`
`1850`	`1850`	`for (int index = 1; index < identifier.length(); index++) {`
`1851`		`- if (!Character.isJavaIdentifierPart(identifier.charAt(index))) {`
	`1851`	`+ if (!Character.isUnicodeIdentifierPart(identifier.charAt(index))) {`
`1852`	`1852`	`return false;`
`1853`	`1853`	`}`
`1854`	`1854`	`}`