From 9f4c2ce5fce8a4226288f2392ea7c427eb55e64c Mon Sep 17 00:00:00 2001
From: Camotoy <20743703+Camotoy@users.noreply.github.com>
Date: Fri, 13 Aug 2021 23:56:44 -0400
Subject: [PATCH] Fix rare StackOverflowError in DeclareCommands

This behavior is exhibited with the NanoLimbo software version 1.2.2.
---
 .../translators/java/JavaDeclareCommandsTranslator.java      | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/connector/src/main/java/org/geysermc/connector/network/translators/java/JavaDeclareCommandsTranslator.java b/connector/src/main/java/org/geysermc/connector/network/translators/java/JavaDeclareCommandsTranslator.java
index 835432376..4780360dd 100644
--- a/connector/src/main/java/org/geysermc/connector/network/translators/java/JavaDeclareCommandsTranslator.java
+++ b/connector/src/main/java/org/geysermc/connector/network/translators/java/JavaDeclareCommandsTranslator.java
@@ -295,6 +295,11 @@ public class JavaDeclareCommandsTranslator extends PacketTranslator<ServerDeclar
             for (int paramID : paramNode.getChildIndices()) {
                 CommandNode paramNode = allNodes[paramID];
 
+                if (paramNode == this.paramNode) {
+                    // Fixes a StackOverflowError when an argument has itself as a child
+                    continue;
+                }
+
                 if (paramNode.getParser() == null) {
                     boolean foundCompatible = false;
                     for (int i = 0; i < children.size(); i++) {