Packets are crashing client

Originally, the code used to have:

Code:java

Field field = NetworkManager.class.getDeclaredField("k");

However, it seems the "k" was changed to "m" since 1.7.2, which is the version that the plugin is currently working for. So I changed it to "m" and all of a sudden, my client crashes whenever the sendMapData function is called in Frame.java.

Frame.java

Code:java

 
public void sendItemMeta(Player player) {
if (this.cachedItemPacket == null) {
EntityItemFrame entity = this.getNMSEntity();
 
ItemStack item = new ItemStack(Material.MAP);
item.setDurability(this.getMapId());
 
net.minecraft.server.v1_7_R3.ItemStack nmsItem = CraftItemStack.asNMSCopy(item);
nmsItem.count = 1;
nmsItem.a(entity);
 
DataWatcher watcher = new DataWatcher(entity);
watcher.a(2, 5);
watcher.a(3, Byte.valueOf((byte)0));
watcher.watch(2, nmsItem);
watcher.h(2);
 
this.cachedItemPacket = new PacketPlayOutEntityMetadata(entity.getId(), watcher, false);
}
 
if (player != null)
((CraftPlayer)player).getHandle().playerConnection.sendPacket(this.cachedItemPacket);
}
 
public void sendMapData(Player player) {
if (this.cachedDataPacket == null) {
this.cachedDataPacket = new PacketPlayOutMap[128];
 
RenderData data = this.getRenderData();
for (int x = 0; x < 128; x++) {
byte[] bytes = new byte[''];
bytes[1] = ((byte)x);
for (int y = 0; y < 128; y++) {
bytes[(y + 3)] = data.buffer[(y * 128 + x)];
}
 
this.cachedDataPacket[x] = new PacketPlayOutMap(this.getMapId(), bytes);
}
}
 
if (player != null)
this.sendPacketsFast(player, this.cachedDataPacket);
}
 
public void sendPacketsFast(Player player, Packet[] packets) {
try {
NetworkManager netty = ((CraftPlayer)player).getHandle().playerConnection.networkManager;
Field field = NetworkManager.class.getDeclaredField("m");
field.setAccessible(true);
Channel channel = (Channel)field.get(netty);
 
for (Packet packet : packets) {
channel.write(packet);
}
channel.flush();
} catch (Exception e) {
FramePicturePlugin.log.log(Level.WARNING, "Cant't send packets!", e);
}
}

FakeEntityTrackerEntry.java (calls functions in Frame.java):

Code:java

@Override
public void updatePlayer(EntityPlayer entityplayer) {
if (!(this.tracker instanceof EntityItemFrame)) {
super.updatePlayer(entityplayer);
return;
}
 
double d0 = entityplayer.locX - this.xLoc / 32;
double d1 = entityplayer.locZ - this.zLoc / 32;
 
if ((d0 >= -this.b) && (d0 <= this.b) && (d1 >= -this.b) && (d1 <= this.b)) {
if ((!this.trackedPlayers.contains(entityplayer)) && ((d(entityplayer)) || (this.tracker.n))) {
EntityItemFrame entity = (EntityItemFrame) this.tracker;
Location loc = this.createLocation(entity);
Frame frame = FramePicturePlugin.getManager().getFrame(loc);
 
if (frame == null) {
super.updatePlayer(entityplayer);
return;
}
entityplayer.removeQueue.remove(Integer.valueOf(this.tracker.getId()));
 
this.trackedPlayers.add(entityplayer);
Player player = entityplayer.getBukkitEntity();
Packet packet = this.createPacket();
 
entityplayer.playerConnection.sendPacket(packet);
frame.sendMapData(player);
frame.sendItemMeta(player);
}
} else if (this.trackedPlayers.contains(entityplayer)) {
this.trackedPlayers.remove(entityplayer);
entityplayer.removeQueue.add(Integer.valueOf(this.tracker.getId()));
}
}
 
public Packet createPacket() {
if (this.tracker.dead || !(this.tracker instanceof EntityItemFrame)) {
return null;
}
EntityItemFrame entity = (EntityItemFrame)this.tracker;
 
PacketPlayOutSpawnEntity packet = new PacketPlayOutSpawnEntity(this.tracker, 71, entity.direction);
packet.a(MathHelper.d(entity.x * 32));
packet.b(MathHelper.d(entity.y * 32));
packet.c(MathHelper.d(entity.z * 32));
return packet;
}

The issue is, when the two snippets above work together (the bottom code snippet initiates the functions in the first snippet), my client crashes:

Description: Ticking screen

 

compare source codes of current version with prev version, NMS coding cannot be done by random.

 

Sorry, but I am an idiot at this. I've been changing a plugin (not mine) to work with 1.7.8 but this is the last issue I am experiencing.
Could you guide me a little further?

Also, not sure if this is useful but this is found in Frame.java as well:

Code:java

 public RenderData getRenderData() {
RenderData render = new RenderData();
MapRenderer mapRenderer = this.generateRenderer();
 
Arrays.fill(render.buffer, (byte)0);
render.cursors.clear();
 
Player player = (Bukkit.getOnlinePlayers().length == 0) ? null : Bukkit.getOnlinePlayers()[0];
FakeMapCanvas canvas = new FakeMapCanvas();
canvas.setBase(render.buffer);
mapRenderer.render(canvas.getMapView(), canvas, player);
 
byte[] buf = canvas.getBuffer();
for (int i = 0; i < buf.length; i++) {
byte color = buf[I];[/I]
if ((color >= 0) || (color <= -113)) render.buffer = color;
}
 
return render;
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Slendemic
Well, just send it the normal way instead of sending through netmanager

 

Slendemic
If you can't compare two versions of same class you shoud not ever try to code.

compare classes and post results.

 

I've compared NetworkManager.java from 1.7.2, 1.7.5, and 1.7.8. I honestly see nothing that would break the code. The issue occurs when sendMapData is called and I am upgrading the plugin from 1.7.2. I still see nothing.

 

Still not resolved.