GrimAnticheat / Grim

Fully async, multithreaded, predictive, open source, 3.01 reach, 1.005 timer, 0.01% speed, 99.99% antikb, "bypassable" 1.8-1.20 anticheat.
GNU General Public License v3.0
1.11k stars 329 forks source link

BADPACKETSE false for 1.21.2+ clients #1769

Open SergioK29 opened 1 month ago

SergioK29 commented 1 month ago

Describe the false positive and how to replicate it

Ride any vehicle in 1.21.2 (using viaversion, not a viaversion issue though, I think they made clients send flying packets while riding in 1.21.2+, or changed how steer vehicle packets are sent, idk)

Grim version

b0cd32622e0dbaa1d6bded568e4db80d1ec21d6d a bit out of date but im pretty sure same issue happens on latest grim

Server version

This server is running Paper version 1.21.1-DEV-master@14a48cd (2024-10-21T13:11:50Z) (Implementing API version 1.21.1-R0.1-SNAPSHOT) You are 1 version(s) behind Download the new version at: https://papermc.io/downloads/paper Previous version: 1.21.1-DEV-7cd4f2c (MC: 1.21.1)

Client version

1.21.3

Plugins

Grim

SamB440 commented 1 month ago

ViaVersion has decided it’s a great idea to send steer vehicle packets when a player is not in a vehicle.

We are done trying to fix via issues. We will be making a packetevents fork and bypassing via for certain listeners.

SamB440 commented 1 month ago

As a temporary fix we can see if they are in a vehicle server side

SergioK29 commented 1 month ago

As a temporary fix we can see if they are in a vehicle server side

turns out it also falses even with vehicle check im pretty sure, maybe they just left vehicle but idk, im not using grims lag compenated vehicle check just player.isInVehicle()

SamB440 commented 1 month ago

Bukkit player? Try the compensatedEntities.serverPlayerVehicle or whatever it was called.

SergioK29 commented 1 month ago

Bukkit player? Try the compensatedEntities.serverPlayerVehicle or whatever it was called.

well currently I just have it disabled for now until i can get rid of viaversion

ManInMyVan commented 1 month ago

for certain listeners

why not all of them?