Skip to content

Commit d06f775

Browse files
Machine-Makerlynxplay
authored andcommitted
early out in getAssetObfuscation
1 parent 94fec5b commit d06f775

File tree

4 files changed

+7
-5
lines changed

4 files changed

+7
-5
lines changed

paper-server/patches/sources/net/minecraft/network/protocol/game/ClientboundSetEntityDataPacket.java.patch

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
}
55

66
private static void pack(List<SynchedEntityData.DataValue<?>> dataValues, RegistryFriendlyByteBuf buffer) {
7-
+ try (io.papermc.paper.util.ItemObfuscationSession ignored = io.papermc.paper.util.ItemObfuscationSession.start(io.papermc.paper.util.ItemObfuscationBinding.LEVEL)) { // Paper - data sanitization
7+
+ try (io.papermc.paper.util.ItemObfuscationSession ignored = io.papermc.paper.util.ItemObfuscationSession.start(io.papermc.paper.configuration.GlobalConfiguration.get().anticheat.obfuscation.items.binding.level)) { // Paper - data sanitization
88
for (SynchedEntityData.DataValue<?> dataValue : dataValues) {
99
dataValue.write(buffer);
1010
}

paper-server/patches/sources/net/minecraft/network/protocol/game/ClientboundSetEquipmentPacket.java.patch

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
buffer.writeVarInt(this.entity);
1919
int size = this.slots.size();
2020

21-
+ try (final io.papermc.paper.util.ItemObfuscationSession ignored = io.papermc.paper.util.ItemObfuscationSession.start(this.sanitize ? io.papermc.paper.util.ItemObfuscationBinding.LEVEL : io.papermc.paper.util.ItemObfuscationSession.ObfuscationLevel.NONE)) { // Paper - data sanitization
21+
+ try (final io.papermc.paper.util.ItemObfuscationSession ignored = io.papermc.paper.util.ItemObfuscationSession.start(this.sanitize ? io.papermc.paper.configuration.GlobalConfiguration.get().anticheat.obfuscation.items.binding.level : io.papermc.paper.util.ItemObfuscationSession.ObfuscationLevel.NONE)) { // Paper - data sanitization
2222
for (int i = 0; i < size; i++) {
2323
Pair<EquipmentSlot, ItemStack> pair = this.slots.get(i);
2424
EquipmentSlot equipmentSlot = pair.getFirst();

paper-server/src/main/java/io/papermc/paper/configuration/GlobalConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -391,7 +391,7 @@ public class Items extends ConfigurationPart {
391391

392392
@PostProcess
393393
public void bindDataSanitizer() {
394-
this.binding = new ItemObfuscationBinding(GlobalConfiguration.this);
394+
this.binding = new ItemObfuscationBinding(this);
395395
}
396396
}
397397
}

paper-server/src/main/java/io/papermc/paper/util/ItemObfuscationBinding.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ public final class ItemObfuscationBinding {
2626
private final BoundObfuscationConfiguration base;
2727
private final Map<ResourceLocation, BoundObfuscationConfiguration> overrides;
2828

29-
public ItemObfuscationBinding(final GlobalConfiguration config) {
30-
final GlobalConfiguration.Anticheat.Obfuscation.Items items = config.anticheat.obfuscation.items;
29+
public ItemObfuscationBinding(final GlobalConfiguration.Anticheat.Obfuscation.Items items) {
3130
this.level = items.enableItemObfuscation ? ItemObfuscationSession.ObfuscationLevel.ALL : ItemObfuscationSession.ObfuscationLevel.OVERSIZED;
3231
this.base = bind(items.allModels);
3332
final Map<ResourceLocation, BoundObfuscationConfiguration> overrides = new HashMap<>();
@@ -79,6 +78,9 @@ private static BoundObfuscationConfiguration bind(final AssetObfuscationConfigur
7978
}
8079

8180
public BoundObfuscationConfiguration getAssetObfuscation(final ItemStack itemStack) {
81+
if (this.overrides.isEmpty()) {
82+
return this.base;
83+
}
8284
return this.overrides.getOrDefault(itemStack.get(DataComponents.ITEM_MODEL), this.base);
8385
}
8486

0 commit comments

Comments
 (0)