Avoid that empty method could be followed by another frame of exit advice that would be added at the same offset and therefore break an assumption on that two frames cannot be written to the same offset. Instead, write a padding NOP byte code when a method returns void to substitute for the single RETURN statement.

Author: raphw

byte-buddy-dep/src/main/java/net/bytebuddy/asm/Advice.java

@@ -10710,7 +10710,9 @@ public void apply() {
                             methodVisitor.visitVarInsn(Opcodes.FSTORE, isExitAdvice() ? argumentHandler.exit() : argumentHandler.enter());
                         } else if (adviceMethod.getReturnType().represents(double.class)) {
                             methodVisitor.visitVarInsn(Opcodes.DSTORE, isExitAdvice() ? argumentHandler.exit() : argumentHandler.enter());
-                        } else if (!adviceMethod.getReturnType().represents(void.class)) {
+                        } else if (adviceMethod.getReturnType().represents(void.class)) {
+                            methodVisitor.visitInsn(Opcodes.NOP);
+                        } else {
                             methodVisitor.visitVarInsn(Opcodes.ASTORE, isExitAdvice() ? argumentHandler.exit() : argumentHandler.enter());
                         }
                         methodSizeHandler.requireStackSize(postProcessor

byte-buddy-dep/src/main/java/net/bytebuddy/utility/visitor/StackAwareMethodVisitor.java

@@ -411,11 +411,6 @@ public void visitLabel(Label label) {
         super.visitLabel(label);
     }
 
-    @Override
-    public void visitLineNumber(int line, Label start) {
-        super.visitLineNumber(line, start);
-    }
-
     @Override
     public void visitTableSwitchInsn(int minimum, int maximum, Label defaultOption, Label... option) {
         adjustStack(-1);