[ASM] Генерация локальной переменной

[Angelok]

Доброй ночи всем. Нужна мелкая помощь по asm`у. Вообщем, очень часто бывают случаи, где нужно "запаковать" значение в локальную переменную. Допустим чтобы вставить код до RETURN`а / THROW`a и не только. Так вот, чтобы "создать" эту переменную я всегда пользовался следующим кодом, который сам же написал, и как ни странно, он работал нормально:

    public static LocalVariableNode createLocalField(MethodNode method, String descriptor) {
        var label = new LabelNode();
        if (method.localVariables == null)
            method.localVariables = new ArrayList();
        var defaultName = "varASM_";
        var suffix = method.localVariables.stream().filter(var -> var.name.startsWith(defaultName)).map((varx) -> varx.name.substring(defaultName.length())).mapToInt(ASMUtils::parseInt).max().orElse(-1) + 1;
        var var = new LocalVariableNode(defaultName + suffix, descriptor, (String)null, label, label, method.localVariables.size());
        method.localVariables.add(var);
        return var;
    }

    private static int parseInt(String str) {
        try {
            return Integer.parseInt(str);
        } catch (Exception var2) {
            return 0;
        }
    }

Но по неизвестным причинам этот способ перестал работать и начал давать ошибку при использовании. Возможно беда в флагах для ClassReader и(или) ClassWriter. Для них я в последнее время юзаю примерно такую логику, не знаю на сколько она верна, ибо в гугле нет инфы, как определить, какие флаги нужно сюда пихать, по этому если знаете как правильнее - напишите. Текущий код:

public byte[] transform(String name, String transformedName, byte[] clazzBytes) {
        ClassNode classNode = new ClassNode();
        int majorVersion = (clazzBytes[6] & 255) << 8 | clazzBytes[7] & 255;
        boolean java7 = majorVersion > 50;
        ClassReader classReader = new ClassReader(clazzBytes);
        classReader.accept(classNode, java7 ? 4 : 8);
        //Сам трансформер
        ClassWriter writer = new ClassWriter(java7 ? 2 : 1);
        classNode.accept(writer);
        return writer.toByteArray();
    }

Основной же мой вопрос - как "доделать" метод генерации локальных переменных в методе, чтобы не было подобных ошибок (краш ниже)?

 

[Angelok]

Вопрос не актуален. Проблема оказалась не в генерации переменной, а в том, что я вставил несколько инструкций подряд через method.instructions.insert(), а нужно было сначала их собрать в InsnList, и уже потом его вставлять.

 

[TheAndrey]

Впервые вижу такую мудрёную логику чтения и записи класса. В чём смысл определять версию класса? Использовать числовые значения вместо констант в исходнике - это не комильфо.

 

[tox1cozZ]

В чём смысл определять версию класса?

Начиная с седьмой версии джавы, сильно изменился процесс верификации байткода.
Ради этого приходится включать автоматическую генерацию stack map frame'ов.
На более старых версиях байткода это лишняя трата времени.
Подробнее здесь: What is a stack map frame

 

[TheAndrey]

Начиная с седьмой версии джавы, сильно изменился процесс верификации байткода.

А в коде упоминается какое-то различие между седьмой и восьмой. В принципе, библиотека сама должна с этим справляться, используя фиксированно заданный набор флагов без лишнего if.

 

[hohserg]

Попробуй не учитывать этот аспект, поделись результатами

 

[tox1cozZ]

А в коде упоминается какое-то различие между седьмой и восьмой. В принципе, библиотека сама должна с этим справляться, используя фиксированно заданный набор флагов без лишнего if.

Ты не видишь что-ли что там тернарный оператор?