Merge pull request #5690 from bjhargrave/issues/5689

classfile: Properly handle unnamed MethodParameter
bndtools · Jun 9, 2023 · d31e205 · d31e205
2 parents 5c2955e + 3997b2e
commit d31e205
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/biz.aQute.bnd.util/src/aQute/bnd/classfile/MethodParametersAttribute.java b/biz.aQute.bnd.util/src/aQute/bnd/classfile/MethodParametersAttribute.java
@@ -71,11 +71,12 @@ public String toString() {
 		static MethodParameter read(DataInput in, ConstantPool constant_pool) throws IOException {
 			int name_index = in.readUnsignedShort();
 			int access_flags = in.readUnsignedShort();
-			return new MethodParameter(constant_pool.utf8(name_index), access_flags);
+			String name = (name_index == 0) ? null : constant_pool.utf8(name_index);
+			return new MethodParameter(name, access_flags);
 		}
 
 		void write(DataOutput out, ConstantPool constant_pool) throws IOException {
-			int name_index = constant_pool.utf8Info(name);
+			int name_index = (name == null) ? 0 : constant_pool.utf8Info(name);
 			out.writeShort(name_index);
 			out.writeShort(access_flags);
 		}