MiniDay/Velocity
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Add another validation case although it's not strictly required

Browse Source
This commit is contained in:
Andrew Steinborn
2020-11-24 12:05:27 -05:00
parent 084b741375
commit aa7aee9dd7
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
proxy/src/main/java/com/velocitypowered/proxy/protocol/packet/AvailableCommands.java
View File

@@ -220,13 +220,13 @@ public class AvailableCommands implements MinecraftPacket {
// Ensure all children exist. Note that we delay checking if the node has been built yet; // Ensure all children exist. Note that we delay checking if the node has been built yet;
// that needs to come after this node is built. // that needs to come after this node is built.
for (int child : children) { for (int child : children) {
if (child >= wireNodes.length) { if (child < 0 || child >= wireNodes.length) {
throw new IllegalStateException("Node points to non-existent index " + redirectTo); throw new IllegalStateException("Node points to non-existent index " + redirectTo);
} }
} }
if (redirectTo != -1) { if (redirectTo != -1) {
if (redirectTo >= wireNodes.length) { if (redirectTo < 0 || redirectTo >= wireNodes.length) {
throw new IllegalStateException("Node points to non-existent index " + redirectTo); throw new IllegalStateException("Node points to non-existent index " + redirectTo);
} }
} }