baritone icon indicating copy to clipboard operation
baritone copied to clipboard

Instant crash upon changing layer order while paused

Open silo8811 opened this issue 1 year ago • 1 comments

Some information

Operating system: Win 11 build 22631 Java version: 23 Minecraft version: 1.20.4 Baritone version: 1.10.2 Other mods (if used): Cloth config api, fabric api (ofc) gamma utils, malilib

Exception, error or logs

Please find your latest.log or debug.log in this folder and attach it to the issue

Note I was on a server so I have (attempted) to censor out information not pertaining to the bug itsself

2024-10-01-1.log

How to reproduce

Step 1: Get yourself tools. A simple pick and shovel. Step 2: Go to one random coordinate on the world and do sel 1 Step 3: Go to somewhere nearby but not too far. Then go underground and do sel 2 Step 4: Tell baritone set buildInLayers true Step 5: Start baritone with sel ca Step 6: Pause baritone with paws Step 7: Tell baritone set layerOrder true (assuming it was false) Step 8: Crash the game by doing unpaws

Modified settings

image

Final checklist

  • [x] I know how to properly use check boxes
  • [x] I have included the version of Minecraft I'm running, baritone's version and forge mods (if used).
  • [x] I have included logs, exceptions and / or steps to reproduce the issue.
  • [x] I have not used any OwO's or UwU's in this issue.

silo8811 avatar Oct 01 '24 16:10 silo8811

Looks like (another case of) assuming an invariant which doesn't actually hold. If I correctly guessed to unobfuscated stack trace this is caused by code assuming that incorrectPositions only contains in-bounds positions while in reality the bounds might have changed since the bounds check.

ZacSharp avatar Oct 03 '24 21:10 ZacSharp