Upstream/NoCheatPlus
1
0
Fork 0
mirror of https://github.com/NoCheatPlus/NoCheatPlus.git synced 2024-10-06 10:27:26 +02:00
Code Issues Projects Releases Wiki Activity

Extend margin for PaperSpigot workaround, confine by split move 1.

Browse Source 
Add a tag.
This commit is contained in:
asofold 2015-12-09 22:14:37 +01:00
parent 4e0b617bea
commit f9f04feee0
2 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java
View File

@ -663,7 +663,7 @@ public class MovingListener extends CheckListener implements TickListener, IRemo
// Actual check. // Actual check.
if (newTo == null) { if (newTo == null) {
// Only check if passable has not already set back. // Only check if passable has not already set back.
newTo = survivalFly.check(player, pFrom, pTo, isSamePos, moveInfo.data, data, cc, time); newTo = survivalFly.check(player, pFrom, pTo, isSamePos, mightBeMultipleMoves, moveInfo.data, data, cc, time);
} }
// Only check NoFall, if not already vetoed. // Only check NoFall, if not already vetoed.
if (checkNf) { if (checkNf) {

15
NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/SurvivalFly.java
View File

@ -121,7 +121,7 @@ public class SurvivalFly extends Check {
* @param isSamePos * @param isSamePos
* @return the location * @return the location
*/ */
public Location check(final Player player, final PlayerLocation from, final PlayerLocation to, final boolean isSamePos, final MoveData moveData, final MovingData data, final MovingConfig cc, final long now) { public Location check(final Player player, final PlayerLocation from, final PlayerLocation to, final boolean isSamePos, final boolean mightBeMultipleMoves, final MoveData moveData, final MovingData data, final MovingConfig cc, final long now) {
tags.clear(); tags.clear();
// Calculate some distances. // Calculate some distances.
@ -370,7 +370,7 @@ public class SurvivalFly extends Check {
} }
} }
else { else {
final double[] res = vDistAir(now, player, from, fromOnGround, resetFrom, to, toOnGround, resetTo, hDistanceAboveLimit, yDistance, data, cc); final double[] res = vDistAir(now, player, from, fromOnGround, resetFrom, to, toOnGround, resetTo, hDistanceAboveLimit, yDistance, mightBeMultipleMoves, data, cc);
vAllowedDistance = res[0]; vAllowedDistance = res[0];
vDistanceAboveLimit = res[1]; vDistanceAboveLimit = res[1];
} }
@ -804,7 +804,7 @@ public class SurvivalFly extends Check {
* Core y-distance checks for in-air movement (may include air -> other). * Core y-distance checks for in-air movement (may include air -> other).
* @return * @return
*/ */
private double[] vDistAir(final long now, final Player player, final PlayerLocation from, final boolean fromOnGround, final boolean resetFrom, final PlayerLocation to, final boolean toOnGround, final boolean resetTo, final double hDistance, final double yDistance, final MovingData data, final MovingConfig cc) { private double[] vDistAir(final long now, final Player player, final PlayerLocation from, final boolean fromOnGround, final boolean resetFrom, final PlayerLocation to, final boolean toOnGround, final boolean resetTo, final double hDistance, final double yDistance, final boolean mightBeMultipleMoves, final MovingData data, final MovingConfig cc) {
// Y-distance for normal jumping, like in air. // Y-distance for normal jumping, like in air.
double vAllowedDistance = 0.0; double vAllowedDistance = 0.0;
double vDistanceAboveLimit = 0.0; double vDistanceAboveLimit = 0.0;
@ -1018,11 +1018,14 @@ public class SurvivalFly extends Check {
if (yDistance > cc.sfStepHeight || !tags.contains("lostground_couldstep")) { if (yDistance > cc.sfStepHeight || !tags.contains("lostground_couldstep")) {
if (data.getOrUseVerticalVelocity(yDistance) == null) { if (data.getOrUseVerticalVelocity(yDistance) == null) {
// Last minute special case. // Last minute special case.
if (data.lastYDist == Double.MAX_VALUE && data.sfJumpPhase == 0 if (data.lastYDist == Double.MAX_VALUE && data.sfJumpPhase == 0 && mightBeMultipleMoves
&& Math.abs(totalVDistViolation) < 0.000000005 && yDistance > 0.0 && yDistance < 0.000000005 && Math.abs(totalVDistViolation) < 0.01 && yDistance > 0.0 && yDistance < 0.01
&& !resetFrom && !resetTo && !data.noFallAssumeGround && !resetFrom && !resetTo && !data.noFallAssumeGround
) { ) {
// Special case after teleport to in-air (PaperSpigot, might confine further by server type/version). // Special case after teleport to in-air (PaperSpigot).
// TODO: Confine to PaperSpigot?
// TODO: Confine to from at block level (offest 0)?
tags.add("skip_paper");
} else { } else {
vDistanceAboveLimit = Math.max(vDistanceAboveLimit, totalVDistViolation); vDistanceAboveLimit = Math.max(vDistanceAboveLimit, totalVDistViolation);
tags.add("vdistsb"); tags.add("vdistsb");