Fixed updatedncp switchitem bypass

previous one fixes switchitem noslow if it is send constantly, but if it send once + c08 packet, previous fix will not work, so this one should patch it
TreemanKing · Aug 30, 2024 · 6143288 · 6143288
1 parent dd2136e
commit 6143288
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 1 deletion.
diff --git a/...atProtocolLib/src/main/java/fr/neatmonster/nocheatplus/checks/net/protocollib/NoSlow.java b/...atProtocolLib/src/main/java/fr/neatmonster/nocheatplus/checks/net/protocollib/NoSlow.java
@@ -216,7 +216,13 @@ private static void onChangeSlot(final PlayerItemHeldEvent e) {
         //    p.getInventory().setHeldItemSlot(data.olditemslot);
         //    data.changeslot = false;
         //}
-        if (e.getPreviousSlot() != e.getNewSlot()) data.isUsingItem = false;
+        if (e.getPreviousSlot() != e.getNewSlot()) {
+            if ((data.isUsingItem || p.isBlocking()) && data.playerMoves.getCurrentMove() != null) {
+                p.getInventory().setHeldItemSlot(e.getPreviousSlot());
+                data.invalidItemUse = true;
+            }
+            data.isUsingItem = false;
+        }
     }
 
     private static boolean hasArrow(final PlayerInventory i, final boolean fw) {
@@ -233,6 +239,7 @@ private void handleBlockPlacePacket(PacketEvent event) {
         final IPlayerData pData = DataManager.getPlayerData(p);
         final MovingData data = pData.getGenericInstance(MovingData.class);
         final PacketContainer packet = event.getPacket();
+
         final StructureModifier<Integer> ints = packet.getIntegers();
         // Legacy: pre 1.9
         if (ints.size() > 0 && !ServerIsAtLeast1_9) {

diff --git a/NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/MovingData.java b/NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/MovingData.java
@@ -157,6 +157,7 @@ public class MovingData extends ACheckData implements IDataOnRemoveSubCheckData,
     public long releaseItemTime = 0;
     /** Detection flag */
     public boolean isHackingRI = false;
+    public boolean invalidItemUse = false;
     /** Keep track of hopping while using items */
     public int noSlowHop = 0;
 

diff --git a/NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/player/SurvivalFly.java b/NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/player/SurvivalFly.java
@@ -1197,6 +1197,15 @@ else if (data.isHackingRI && (!checkPermissions || !pData.hasPermission(Permissi
             friction = 0.0;
         }
 
+        // InvalidUse packet
+        else if (data.invalidItemUse && (!checkPermissions || !pData.hasPermission(Permissions.MOVING_SURVIVALFLY_BLOCKING, player))) {
+            tags.add("invalidate_use");
+            data.invalidItemUse = false;
+            hAllowedDistance = 0.0;
+            useBaseModifiers = false;
+            friction = 0.0;
+        }
+
         // Collision tolerance for entities (1.9+)
         else if (ServerIsAtLeast1_9 && CollisionUtil.isCollidingWithEntities(player, true) 
                 && hAllowedDistance < 0.35 && data.liftOffEnvelope == LiftOffEnvelope.NORMAL) {