From 6f56e1acaf4138c943758db427d5d28ce9d590ab Mon Sep 17 00:00:00 2001 From: miniduikboot Date: Sun, 28 Apr 2024 17:00:30 +0200 Subject: [PATCH] Move remaining Color checks to ColorLimits Some mods add custom colors and still use the standard vanilla SetColor RPC, which leads to issues with this check. --- src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.cs b/src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.cs index 9cfe037..37dffe5 100644 --- a/src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.cs +++ b/src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.cs @@ -688,7 +688,7 @@ namespace Impostor.Server.Net.Inner.Objects if ((byte)color > ColorsCount) { - if (await sender.Client.ReportCheatAsync(RpcCalls.CheckColor, CheatCategory.ProtocolExtension, "Client sent invalid color")) + if (await sender.Client.ReportCheatAsync(RpcCalls.CheckColor, CheatCategory.ColorLimits, "Client sent invalid color")) { return false; } @@ -736,7 +736,7 @@ namespace Impostor.Server.Net.Inner.Objects if (colorOffset == ColorsCount) { - if (await sender.Client.ReportCheatAsync(RpcCalls.SetColor, CheatCategory.GameFlow, "Client sent SetColor but all colors are already in use")) + if (await sender.Client.ReportCheatAsync(RpcCalls.SetColor, CheatCategory.ColorLimits, "Client sent SetColor but all colors are already in use")) { await SetColorAsync(expected); return false; @@ -746,7 +746,7 @@ namespace Impostor.Server.Net.Inner.Objects if (color != expected) { - if (await sender.Client.ReportCheatAsync(RpcCalls.SetColor, CheatCategory.GameFlow, "Client sent SetColor with incorrect color")) + if (await sender.Client.ReportCheatAsync(RpcCalls.SetColor, CheatCategory.ColorLimits, "Client sent SetColor with incorrect color")) { await SetColorAsync(expected); return false; -- 2.39.5