Jump to content
Search the Forums BEFORE making a new post.
Currently supported: All versions

[Solved] Help with ASM

Arlyh

By Arlyh
in Modder Support

 Share

Recommended Posts

Arlyh Tree Puncher

Arlyh

Posted
Posted

Hello, I'm attempting to rewrite a minecraftforge method in net.minecraft.block.Block, 

boolean removeBlockByPlayer(World, EntityPlayer, Int, Int, Int)

 

I start by first removing the original method with a ClassVisitor and an overloaded visitMethod that returns null.  I then try to use a MethodVisitor to rewrite the bytecode.  Unfortunately, it seems to be rewriting the net.minecraft.block.Block constructor rather than the intended function, though I have determined that removeBlockByPlayer is indeed deleted by iterating through the ClassNode.methods list.

 

My code is as follows:

 

 

package servercommands.asm;

import static org.objectweb.asm.ClassWriter.*;
import static org.objectweb.asm.Opcodes.*;

import java.util.List;

import org.objectweb.asm.ClassReader;
import org.objectweb.asm.ClassVisitor;
import org.objectweb.asm.ClassWriter;
import org.objectweb.asm.Label;
import org.objectweb.asm.MethodVisitor;
import org.objectweb.asm.Opcodes;
import org.objectweb.asm.tree.ClassNode;
import org.objectweb.asm.tree.MethodNode;
import org.objectweb.asm.tree.TypeInsnNode;

import cpw.mods.fml.relauncher.IClassTransformer;

public class ServerCommandsAccessTransformer implements IClassTransformer {

public byte[] deleteMethod(byte[] bytes, int access, String name, String desc) {
	ClassWriter cw = new ClassWriter(COMPUTE_FRAMES);
	ClassVisitor cv = new ClassVisitor(Opcodes.ASM4, cw) {
		@Override
		public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) {
			return null;
		}
	};

	ClassReader cr = new ClassReader(bytes);
	cr.accept(cv,  0);
	cv.visitMethod(access, name, desc, null, null);
	return cw.toByteArray();
}

public byte[] transformer001(String name, byte[] bytes) {
	if(name.equals(ServerCommandsLoadingPlugin.debug ? "net.minecraft.block.Block" : "amj")) {
		bytes = deleteMethod(bytes, ACC_PUBLIC, "removeBlockByPlayer", ServerCommandsLoadingPlugin.debug ? "(Lnet/minecraft/world/World;Lnet/minecraft/entity/player/EntityPlayer;III)Z" : "(Lxv;Lqx;III)Z");
		ClassNode node = new ClassNode();
		ClassReader reader = new ClassReader(bytes);
		reader.accept(node, 0);

		ClassWriter cw = new ClassWriter(COMPUTE_FRAMES);
		node.accept(cw);

		MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "removeBlockByPlayer", ServerCommandsLoadingPlugin.debug ? "(Lnet/minecraft/world/World;Lnet/minecraft/entity/player/EntityPlayer;III)Z" : "(Lxv;Lqx;III)Z", null, null);

		mv.visitCode();
		mv.visitTypeInsn(NEW, "servercommands/PlayerBreakBlockEvent");
		mv.visitInsn(DUP);
		mv.visitVarInsn(ALOAD, 1);
		mv.visitVarInsn(ALOAD, 0);
		mv.visitVarInsn(ALOAD, 2);
		mv.visitVarInsn(ILOAD, 3);
		mv.visitVarInsn(ILOAD, 4);
		mv.visitVarInsn(ILOAD, 5);
		mv.visitMethodInsn(INVOKESPECIAL, "servercommands/PlayerBreakBlockEvent", "<init>", ServerCommandsLoadingPlugin.debug ? "(Lnet/minecraft/world/World;Lnet/minecraft/entity/player/EntityPlayer;III)Z" : "(Lxv;Lqx;III)Z");
		mv.visitVarInsn(ASTORE, 6);
		mv.visitFieldInsn(GETSTATIC, "net/minecraftforge/common/MinecraftForge", "EVENT_BUS", "Lnet/minecraftforge/event/EventBus;");
		mv.visitVarInsn(ALOAD, 6);
		mv.visitMethodInsn(INVOKEVIRTUAL, "net/minecraftforge/event/EventBus", "post", "(Lnet/minecraftforge/event/Event;)Z");
		mv.visitInsn(POP);
		mv.visitVarInsn(ALOAD, 6);
		mv.visitFieldInsn(GETFIELD, "servercommands/PlayerBreakBlockEvent", "result", "Lnet/minecraftforge/event/Event$Result;");
		mv.visitFieldInsn(GETSTATIC, "net/minecraftforge/event/Event$Result", "DENY", "Lnet/minecraftforge/event/Event$Result;");
		Label l0 = new Label();
		mv.visitJumpInsn(IF_ACMPNE, l0);
		mv.visitInsn(ICONST_0);
		mv.visitInsn(IRETURN);
		mv.visitLabel(l0);
		mv.visitFrame(Opcodes.F_APPEND,1, new Object[] {"servercommands/PlayerBreakBlockEvent"}, 0, null);
		mv.visitVarInsn(ALOAD, 1);
		mv.visitVarInsn(ILOAD, 3);
		mv.visitVarInsn(ILOAD, 4);
		mv.visitVarInsn(ILOAD, 5);
		mv.visitInsn(ICONST_0);
		if(ServerCommandsLoadingPlugin.debug) mv.visitMethodInsn(INVOKEVIRTUAL, "net/minecraft/world/World", "setBlockWithNotify", "(IIII)Z");
		else mv.visitMethodInsn(INVOKEVIRTUAL, "xv", "e", "(IIII)Z");
		mv.visitInsn(IRETURN);
		mv.visitMaxs(8, 7);
		mv.visitEnd();

		bytes = cw.toByteArray();

		System.out.println("Added PlayerBreakBlockEvent to Block.removeBlockByPlayer()");

	}
	return bytes;
}

@Override
public byte[] transform(String name, byte[] bytes) {
	try {
		if(name.contains(ServerCommandsLoadingPlugin.debug ? "net.minecraft.block.Block" : "amj")) System.out.println(name);
		bytes = transformer001(name, bytes);
	} catch(Exception e) {
		throw new RuntimeException(e);
	}
	return bytes;
}
}

 

 

FML output is as follows:

 

 

2012-12-20 17:09:33 [iNFO] [ForgeModLoader] Forge Mod Loader version 4.5.62.495 for Minecraft 1.4.5 loading
2012-12-20 17:09:33 [iNFO] [sTDOUT] Called
2012-12-20 17:09:33 [iNFO] [sTDOUT] net.minecraft.block.Block
2012-12-20 17:09:33 [iNFO] [sTDOUT] Added PlayerBreakBlockEvent to Block.removeBlockByPlayer()
2012-12-20 17:09:33 [iNFO] [sTDERR] java.lang.reflect.InvocationTargetException
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.reflect.Method.invoke(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at cpw.mods.fml.relauncher.FMLRelauncher.relaunchServer(FMLRelauncher.java:138)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at cpw.mods.fml.relauncher.FMLRelauncher.handleServerRelaunch(FMLRelauncher.java:33)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at net.minecraft.server.MinecraftServer.main(MinecraftServer.java:1374)
2012-12-20 17:09:33 [iNFO] [sTDERR] Caused by: java.lang.NoClassDefFoundError: net/minecraft/block/Block
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at net.minecraft.stats.StatList.initMinableStats(StatList.java:185)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at net.minecraft.stats.StatList.<clinit>(StatList.java:96)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at net.minecraft.server.MinecraftServer.fmlReentry(MinecraftServer.java:1381)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	... 7 more
2012-12-20 17:09:33 [iNFO] [sTDERR] Caused by: java.lang.ClassNotFoundException: net.minecraft.block.Block
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at cpw.mods.fml.relauncher.RelaunchClassLoader.findClass(RelaunchClassLoader.java:141)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.ClassLoader.loadClass(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.ClassLoader.loadClass(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	... 10 more
2012-12-20 17:09:33 [iNFO] [sTDERR] Caused by: java.lang.ClassFormatError: Method "<init>" in class net/minecraft/block/Block has illegal signature "(Lnet/minecraft/world/World;Lnet/minecraft/entity/player/EntityPlayer;III)Z"
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.ClassLoader.defineClass1(Native Method)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.ClassLoader.defineClass(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at java.lang.ClassLoader.defineClass(Unknown Source)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	at cpw.mods.fml.relauncher.RelaunchClassLoader.findClass(RelaunchClassLoader.java:134)
2012-12-20 17:09:33 [iNFO] [sTDERR] 	... 12 more

 

 

I apologize for any silly mistakes I may have made.  I primarily code in C/C++ and have only recently taken an interest in Minecraft modding.

keepcalm Diamond Finder

keepcalm

Posted
Posted

In the future, use spoiler, not hidden ;)

 

Also, judging from what you're trying to do, the best way to do it would be to add a function call, rather than rewriting the entire method. You might want to have a look at the source code for my mod, which does something similar, but uses ItemInWorldManager (so it's server-side only).

 

If you want more help, PM me (I think I've worked out how to do this ASM stuff)

Protip: try and find answers yourself before asking on the forum.

It's pretty likely that there is an answer.

 

Was I helpful? Give me a thank you!

 

 

width=635 height=903http://bit.ly/HZ03zy[/img]

 

 

Tired of waiting for mods to port to bukkit?

use BukkitForge! (now with a working version of WorldEdit!)

Arlyh Tree Puncher

Arlyh

Posted
  • Author
Posted

I was able to come up with a solution to my issue, and I would like to make my solution visible to this community.  It is as follows:

 

 

public class ServerCommandsAccessTransformer implements IClassTransformer {

class SCClassVisitor extends ClassVisitor {
	private String methodName;

	public SCClassVisitor(int api, ClassVisitor cv, String methodName) {
		super(api, cv);
		this.methodName = methodName;
	}

	@Override
	public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) {
		if(name.equals(methodName)) {
			return null;
		}
		return super.visitMethod(access, name, desc, signature, exceptions);
	}		
}

public byte[] deleteMethod(byte[] bytes, int access, String name) {
	ClassWriter cw = new ClassWriter(COMPUTE_FRAMES);
	SCClassVisitor cv = new SCClassVisitor(ASM4, cw, name);
	ClassReader cr = new ClassReader(bytes);
	cr.accept(cv,  0);
	return cw.toByteArray();
}


public byte[] transformer001(String name, byte[] bytes) {
	bytes = deleteMethod(bytes, ACC_PUBLIC, "removeBlockByPlayer");

	ClassWriter cw = new ClassWriter(COMPUTE_FRAMES);
	ClassReader cr = new ClassReader(bytes);
	cr.accept(cw, 0);

	MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "removeBlockByPlayer", ServerCommandsLoadingPlugin.debug ? "(Lnet/minecraft/world/World;Lnet/minecraft/entity/player/EntityPlayer;III)Z" : "(Lxv;Lqx;III)Z", null, null);

	mv.visitCode();
	mv.visitTypeInsn(NEW, "servercommands/PlayerBreakBlockEvent");
	mv.visitInsn(DUP);
	mv.visitVarInsn(ALOAD, 1);
	mv.visitVarInsn(ALOAD, 0);
	mv.visitVarInsn(ALOAD, 2);
	mv.visitVarInsn(ILOAD, 3);
	mv.visitVarInsn(ILOAD, 4);
	mv.visitVarInsn(ILOAD, 5);
	mv.visitMethodInsn(INVOKESPECIAL, "servercommands/PlayerBreakBlockEvent", "<init>", ServerCommandsLoadingPlugin.debug ? "(Lnet/minecraft/world/World;Lnet/minecraft/block/Block;Lnet/minecraft/entity/player/EntityPlayer;III)V" : "(Lxv;Lamj;Lqx;III)V");
	mv.visitVarInsn(ASTORE, 6);
	mv.visitFieldInsn(GETSTATIC, "net/minecraftforge/common/MinecraftForge", "EVENT_BUS", "Lnet/minecraftforge/event/EventBus;");
	mv.visitVarInsn(ALOAD, 6);
	mv.visitMethodInsn(INVOKEVIRTUAL, "net/minecraftforge/event/EventBus", "post", "(Lnet/minecraftforge/event/Event;)Z");
	mv.visitInsn(POP);
	mv.visitVarInsn(ALOAD, 6);
	mv.visitFieldInsn(GETFIELD, "servercommands/PlayerBreakBlockEvent", "result", "Lnet/minecraftforge/event/Event$Result;");
	mv.visitFieldInsn(GETSTATIC, "net/minecraftforge/event/Event$Result", "DENY", "Lnet/minecraftforge/event/Event$Result;");
	Label l0 = new Label();
	mv.visitJumpInsn(IF_ACMPNE, l0);
	mv.visitInsn(ICONST_0);
	mv.visitInsn(IRETURN);
	mv.visitLabel(l0);
	mv.visitFrame(Opcodes.F_APPEND,1, new Object[] {"servercommands/PlayerBreakBlockEvent"}, 0, null);
	mv.visitVarInsn(ALOAD, 1);
	mv.visitVarInsn(ILOAD, 3);
	mv.visitVarInsn(ILOAD, 4);
	mv.visitVarInsn(ILOAD, 5);
	mv.visitInsn(ICONST_0);
	if(ServerCommandsLoadingPlugin.debug) mv.visitMethodInsn(INVOKEVIRTUAL, "net/minecraft/world/World", "setBlockWithNotify", "(IIII)Z"); 
	else mv.visitMethodInsn(INVOKEVIRTUAL, "xv", "e", "(IIII)Z");
	mv.visitInsn(IRETURN);
	mv.visitMaxs(8, 7);
	mv.visitEnd();

	return cw.toByteArray();
}

@Override
public byte[] transform(String name, byte[] bytes) {
	try {
		if(name.equals(ServerCommandsLoadingPlugin.debug ? "net.minecraft.block.Block" : "amj")) {
			System.out.println(name);
			bytes = transformer001(name, bytes);
		}
	} catch(Exception e) {
		throw new RuntimeException(e);
	}
	return bytes;
}
}

 

 

Keepcalm, I did end up going a different direction, but I do appreciate your advice and example code.  It is always helpful to see how other programmers are accomplishing similar problems.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing

Announcements



  • Recently Browsing

    • No registered users viewing this page.

  • Posts

    • M4rKitosFdF
      Items are not stacking correctly

      By M4rKitosFdF · Posted

      The specific issue is that items in my inventory wont stack properly. For instance, if I punch a tree down to collect wood, the first block I collected goes to my hand. So when I punch the second block of wood to collect it, it drops, but instead of stacking with the piece of wood already in my hand, it goes to the second slot in my hotbar instead. Another example is that I'll get some dirt, and then when I'm placing it down later I'll accidentally place a block where I don't want it. When I harvest it again, it doesn't go back to the stack that it came from on my hotbar, where it should have gone, but rather into my inventory. That means that if my inventory is full, then the dirt wont be picked up even though there should be space available in the stack I'm holding. The forge version I'm using is 40.3.0, for java 1.18.2. I'll leave the mods I'm using here, and I'd appreciate it if anybody can point me in the right direction in regards to figuring out how to fix this. I forgot to mention that I think it only happens on my server but I&#39;m not entirely sure. PLEASE HELP ME! LIST OF THE MODS. aaa_particles Adorn AdvancementPlaques AI-Improvements AkashicTome alexsdelight alexsmobs AmbientSounds amwplushies Animalistic another_furniture AppleSkin Aquaculture aquamirae architectury artifacts Atlas-Lib AutoLeveling AutoRegLib auudio balm betterfpsdist biggerstacks biomancy BiomesOPlenty blockui blueprint Bookshelf born_in_chaos Botania braincell BrassAmberBattleTowers brutalbosses camera CasinoCraft cfm (MrCrayfish’s Furniture Mod) chat_heads citadel cloth-config Clumps CMDCam CNB cobweb collective comforts convenientcurioscontainer cookingforblockheads coroutil CosmeticArmorReworked CozyHome CrabbersDelight crashexploitfixer crashutilities Create CreativeCore creeperoverhaul cristellib crittersandcompanions Croptopia CroptopiaAdditions CullLessLeaves curios curiouslanterns curiouslights Curses' Naturals CustomNPCs CyclopsCore dannys_expansion decocraft Decoration Mod DecorationDelightRefurbished Decorative Blocks Disenchanting DistantHorizons doubledoors DramaticDoors drippyloadingscreen durabilitytooltip dynamic-fps dynamiclights DynamicTrees DynamicTreesBOP DynamicTreesPlus Easy Dungeons EasyAnvils EasyMagic easy_npc eatinganimation ecologics effective_fg elevatorid embeddium emotecraft enchantlimiter EnchantmentDescriptions EnderMail engineersdecor entityculling entity_model_features entity_texture_features epicfight EvilCraft exlinefurniture expandability explosiveenhancement factory-blocks fairylights fancymenu FancyVideo FarmersDelight fast-ip-ping FastSuite ferritecore finsandtails FixMySpawnR Forge Middle Ages fossil FpsReducer2 furnish GamingDeco geckolib goblintraders goldenfood goodall H.e.b habitat harvest-with-ease hexerei hole_filler huge-structure-blocks HunterIllager iammusicplayer Iceberg illuminations immersive_paintings incubation infinitybuttons inventoryhud InventoryProfilesNext invocore ItemBorders itemzoom Jade jei (Just Enough Items) JetAndEliasArmors journeymap JRFTL justzoom kiwiboi Kobolds konkrete kotlinforforge lazydfu LegendaryTooltips libIPN lightspeed lmft lodestone LongNbtKiller LuckPerms Lucky77 MagmaMonsters malum ManyIdeasCore ManyIdeasDoors marbledsarsenal marg mcw-furniture mcw-lights mcw-paths mcw-stairs mcw-trapdoors mcw-windows meetyourfight melody memoryleakfix Mimic minecraft-comes-alive MineTraps minibosses MmmMmmMmmMmm MOAdecor (ART, BATH, COOKERY, GARDEN, HOLIDAYS, LIGHTS, SCIENCE) MobCatcher modonomicon mods_optimizer morehitboxes mowziesmobs MutantMonsters mysticalworld naturalist NaturesAura neapolitan NekosEnchantedBooks neoncraft2 nerb nifty NightConfigFixes nightlights nocube's_villagers_sell_animals NoSeeNoTick notenoughanimations obscure_api oculus oresabovediamonds otyacraftengine Paraglider Patchouli physics-mod Pillagers Gun PizzaCraft placeableitems Placebo player-animation-lib pneumaticcraft-repressurized polymorph PrettyPipes Prism projectbrazier Psychadelic-Chemistry PuzzlesLib realmrpg_imps_and_demons RecipesLibrary reeves-furniture RegionsUnexplored restrictedportals revive-me Scary_Mobs_And_Bosses selene shetiphiancore ShoulderSurfing smoothboot
    • OhOhIWontLetUGo
      Forge 1.21 – How can I add a custom HUD Overlay (Item Icon + Text)

      By OhOhIWontLetUGo · Posted

      Hi everyone, I'm currently developing a Forge 1.21 mod for Minecraft and I want to display a custom HUD overlay for a minigame. My goal: When the game starts, all players should see an item/block icon (from the base game, not a custom texture) plus its name/text in the HUD – similar to how the bossbar overlay works. The HUD should appear centered above the hotbar (or at a similar prominent spot), and update dynamically (icon and name change as the target item changes). What I've tried: I looked at many online tutorials and several GitHub repos (e.g. SeasonHUD, MiniHUD), but most of them use NeoForge or Forge versions <1.20 that provide the IGuiOverlay API (e.g. implements IGuiOverlay, RegisterGuiOverlaysEvent). In Forge 1.21, it seems that neither IGuiOverlay nor RegisterGuiOverlaysEvent exist anymore – at least, I can't import them and they are missing from the docs and code completion. I tried using RenderLevelStageEvent as a workaround but it is probably not intended for custom HUDs. I am not using NeoForge, and switching the project to NeoForge is currently not an option for me. I tried to look at the original minecraft source code to see how elements like hearts, hotbar etc are drawn on the screen but I am too new to Minecraft modding to understand. What I'm looking for: What is the correct way to add a custom HUD element (icon + text) in Forge 1.21, given that the previous overlay API is missing? Is there a new recommended event, callback, or method in Forge 1.21 for custom HUD overlays, or is everyone just using a workaround? Is there a minimal open-source example repo for Forge 1.21 that demonstrates a working HUD overlay without relying on NeoForge or deprecated Forge APIs? My ideal solution: Centered HUD element with an in-game item/block icon (from the base game's assets, e.g. a diamond or any ItemStack / Item) and its name as text, with a transparent background rectangle. It should be visible to the players when the mini game is running. Easy to update the item (e.g. static variable or other method), so it can change dynamically during the game. Any help, code snippets, or up-to-date references would be really appreciated! If this is simply not possible right now in Forge 1.21, it would also help to know that for sure. Thank you very much in advance!
    • Brown
      [1.20.1] Severe server freeze when exiting Nether portal or entering End (no mods, Forge 1.20.1, Arch Linux)

      By Brown · Posted

      Chunk loading maybe stalls the JVM during dimension travel?
    • Ugdhar
      [Help] How to update a mod from 1.19.2 to 1.20.1

      By Ugdhar · Posted

      The simple answer is there is not an easy way. You would need to know how to program in Java, as well as at least some familiarity with how Forge works so you could port the differences. You would also need the sourcecode for the original mod, and permission from the author to modify it, if they did not use some sort of open source license. So it's not impossible, but it would take some effort, but doing so would open up a whole new world of possibilities for you!
    • Ugdhar
      Crash the server

      By Ugdhar · Posted

      Does it still crash if you remove holdmyitems? Looks like that mod doesn't work on a server as far as I can tell from the error.  

  • Topics

  • Who's Online (See full list)

    • There are no registered users currently online
×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.

  I accept