Jump to content

[1.7.10][Solved] Strange nullPointer when handling a packet (on the client)


Recommended Posts

Posted

For my current mod I need to process data on the client, and send the results back to the server. I am using a system of packets to achieve this, however, I'm not sure what's the best/most efficient way to achieve my goal.

 

The first time any client connects to a server, I will send a packet/packets containing the data I need that client to process. After its done I send the result(s) back to the server via another packet/other packets. On the server I will save these results and after that point no more packets will be sent.

 

As you can probably tell from the way I describe my problem I am not sure what's the most efficient way to achieve this.

 

This is where my question comes in: If you need to process (a lot) of data, is it better to send one big packet, containing all the data which has to be processed, and send a big packet back containing all the results, or is it better to send little packets which each contain a single piece of the data, and send little packets back containing the individual results?

 

my current code crashes every time I join a (new) world, I think this crash is related to my question because i am currently using a mix of big and little packets...

but if you need a (not yet working) piece of my code, i will happily supply it.

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted

1. what type of data are we talking about? clients should not process data, because then they can fake the results..

2. one big packet is the best in my opinion, because theres really no sense in splitting those up

3. it wont crash because u use a mix of packets, itll crash if u mess sth up :b

Posted
  On 8/12/2015 at 12:10 AM, Failender said:

1. what type of data are we talking about? clients should not process data, because then they can fake the results..

2. one big packet is the best in my opinion, because theres really no sense in splitting those up

3. it wont crash because u use a mix of packets, itll crash if u mess sth up :b

 

1. I need a list of all possible items on the server, to achieve this I'm iterating through the itemRegistry, and if an Item has subItems, i need to get those, otherwise i'm just adding a stack with damagevalue 1. the Item.getSubItems() method is ClientSide only, and this is why i need to process this data on the client.

 

2. This is what i needed to know, I am currently sending one big packet from the server to the client, and i am responding with a lot of little packets, but i will change this out to one big packet as well.

 

3. You are right ;) . The crash is happening on the client when handeling the packet. It's a really strange nullPointer and i thought that data maybe got lost because i was using such big packets to the client. Since this is the way to go, this clearly doesn't cause my problems, and i will post more info about the crash in the next post, because i can't seem to figure out the cause myself...

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted

1. why . forge is already kinda doing that  for you with just saying mods need to be on the same versions, or they get a connection refuse. I might be tripping with understanding you there

Posted

This is the code I am currently using for my packet from server to client:

https://gist.github.com/wesserboy/dc7f1f88afb3b83aaac6

(In this code i am still sending multiple small packets to the server, I am changing that as we speak, however this can not be related to the issue, since the code crashes before this is done)

 

This is the stackTrace of the nullPointer:

http://pastebin.com/cnGGaVuV

(from what i can tell it's the same crash happening three times in a row, that's why i've added spaces in between the three individual stackTraces)

 

Line 76 (The for loop in the handleClientSide() method) crashes with a nullPointer.

 

I started to trace this variable backwards (that's why there's a lot of System.out.println()) and this is what i found:

 

on line 74 (the beginning of the handleClientSide() method) the items field is null, this is obviously what causes the nullPointer.

However, on line 46 (the end of the fromBytes() method) the items field is not null...

 

I am not quite sure how packets work behind the scenes, but to my knowledge these methods should follow each other with no changes made in between...

Also, it seems very odd to me that a stackTrace refers to a package declaration... (line 4 of the stackTrace)

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted
  On 8/12/2015 at 12:42 AM, Failender said:

1. why . forge is already kinda doing that  for you with just saying mods need to be on the same versions, or they get a connection refuse. I might be tripping with understanding you there

 

My idea is to make a mod which assigns a random item to each player to acquire, to give the player something to work towards whilst playing.

I need to assign this itemStack on the server (for obvious reasons).

It would be silly if the mod would only ever assign bonemeal, instead of all possible dyes, and only oak wood instead of all types of planks/logs.

This is why i need to index the subItems of these items.

 

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted

after a good night of rest i have figured it out...

It was an incredibly stupid mistake.

 

In my handleClientSide method I was using the field of the object handleClientSide was being called from, instead of the field of the message parameter.  :-[

 

Moral of the story: Don't code if you're tired, it can only end badly...

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted

Good for you.  Still at a loss for why you are sending all that to the client.

 

Decide your item on the server.  Send a chatmessage to the player or just send the one item they need to get.

Long time Bukkit & Forge Programmer

Happy to try and help

Posted
  On 8/12/2015 at 12:54 PM, delpi said:

Good for you.  Still at a loss for why you are sending all that to the client.

 

Decide your item on the server.  Send a chatmessage to the player or just send the one item they need to get.

 

This is what I am doing after the list is constructed, I only do this the first time any client joins, because i need a clientSide only method to construct the item list.

I store this list on the server and after that point any other clients joining will not get this packet.

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted
  On 8/12/2015 at 2:16 PM, diesieben07 said:

One thing: Hacked clients. Your mod is a security hole the size of texas :D

 

I am not quite sure how hacked clients work and what exploits they are using to achieve what they're doing, but is there a way for me to make my mod more secure?

Maybe general rules to stick to? Or some kind of technique I am unfamiliar with?

Preferably something that doesn't involve me having to rewrite my list construction (again), unless it's really necessary...

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted
  On 8/12/2015 at 3:07 PM, diesieben07 said:

As soon as you in any way trust the client you have a security hole. Your mod as you described it completely trusts the client in giving you all the subtypes of an Item. There is nothing stopping a client to tell you that Blocks.stone has an ItemStack of command block with NBT data on it as a subtype. Your mod would happily accept that and bam, people have control over the server without any sort of security mechanism stopping them.

 

I am checking if an item actually has subItems on the server (using item.getHasSubtypes(), which strangely enough isn't ClientSide only), before i add it to the list that is being sent to the client, and I have a blackList system in the config file, which gives servers more control over what items end up in the list, but other than that you are totally right...

 

I could maybe implement a system that checks whether the items of the ItemStacks were actually sent to the client, as an extra security measure.

 

However, I am not sure if it's worth the effort, I am only using this list of itemStacks to assign a 'ChallengeStack' to the player, the player should then try to acquire this itemStack.

I am checking this in the ItemPickupEvent, ItemCraftedEvent and ItemSmeltedEvent. If the itemStack the player picked up/crafted/smelted is equal to the assigned itemStack the player gets a message notifying him of completing the challenge, and a soundeffect starts playing. After that i just assign a new ItemStack as a new challenge.

 

Since I am never giving the player any of these itemStacks, never spawning them in the world and never placing blocks, I think the level of control the client gets is very little, which is why i don't think it's that big of a problem.

but if I missed something, be sure to tell me ;)

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

Posted

Just get a list of all the items on the server (won't have sub), iterate through it, check for subtypes, append the list. 

 

No real need to go to the client for all that.

Long time Bukkit & Forge Programmer

Happy to try and help

Posted
  On 8/12/2015 at 4:58 PM, delpi said:

Just get a list of all the items on the server (won't have sub), iterate through it, check for subtypes, append the list. 

 

No real need to go to the client for all that.

 

ahh, I think I understand what you mean now, this will however not work:

You can get the boolean to know if an item has subtypes on the server, you can however not get those subtypes on the server, this is why I request those subtypes from the client.

 

This is another approach i thought of which is similar to yours, which also doesn't involve as much client data on the server:

- Create a list of all items on the server. (excl. subitems)

- Get a random item to assign.

- If the item has subitems, send it to the client --> let the client generate a random subItem and send it back to the server --> assign itemstack as challenge.

- else just make a new itemStack with damagevalue 0 and assign this to the player. (no client interaction at all)

 

This would not work as i would like it to and i will tell you why:

First you get a random item, if this item doesn't have subitems it's immediately assigned, so the chance to get that itemStack as a challenge is 1 / amount_of_items.

if it does have subItems, one subitem is randomized, so the chance of getting that itemStack as a challenge is (1 / amount_of_items) * (1 / amount_of_subItems) = 1 / (items * subItems).

 

This would make it so some itemStacks have a higher chance of being the challenge than others.

 

A way to fix this would be to add the items that have subItems once to the list for every subItem.

then the chance of getting a certain subItem would be subItems / items * subItems = 1 / items (the same as all the other items)

But as far as i know there is no way to get the amount of subitems an item has, and this makes it so this approach would not work (as intended).

I made the Mob Particles mod, you can check it out here: http://www.minecraftforum.net/topic/2709242-172-forge-mob-particles/

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.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Cracked Launchers are not supported
    • After some time minecraft crashes with an error. Here is the log https://drive.google.com/file/d/1o-2R6KZaC8sxjtLaw5qj0A-GkG_SuoB5/view?usp=sharing
    • 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'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
    • 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!
  • Topics

×
×
  • Create New...

Important Information

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