GeyserMC-Mirror/Geyser
1
0
Fork 0
mirror of https://github.com/GeyserMC/Geyser.git synced 2025-12-19 14:59:27 +00:00
Code Issues Releases Activity

Fix trailing formatting characters causing OOB (#5990)

Browse Source 
* Fix trailing formatting characters causing OOB

* Fix typo
This commit is contained in:
rtm516
2025-11-20 14:05:19 +00:00
committed by GitHub
parent 53596d05bc
commit 846e0b5ce0
2 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
core/src/main/java/org/geysermc/geyser/translator/text/MessageTranslator.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 2019-2022 GeyserMC. http://geysermc.org
* Copyright (c) 2019-2025 GeyserMC. http://geysermc.org
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
@@ -253,6 +253,12 @@ public class MessageTranslator {
output.append(c);
if (c == ChatColor.ESCAPE) {
// If the string ends with a formatting character, remove and skip
if (i >= finalLegacyString.length() - 1) {
output = output.deleteCharAt(output.length() - 1);
continue;
}
char newColor = finalLegacyString.charAt(i + 1);
if (newColor == 'r') {
lastColors = new StringBuilder();

5
core/src/test/java/org/geysermc/geyser/network/translators/chat/MessageTranslatorTest.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 2019-2022 GeyserMC. http://geysermc.org
* Copyright (c) 2019-2025 GeyserMC. http://geysermc.org
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
@@ -73,6 +73,9 @@ public class MessageTranslatorTest {
messages.put("{\"translate\":\"tt{''{tt\"}", "tt{''{tt");
messages.put("{\"translate\":\"tt{{''}}tt\"}", "tt{{''}}tt");
messages.put("{\"text\":\"\",\"extra\":[{\"text\":\"Testing end of string\n formatting character§\",\"color\":\"yellow\"}]}",
"§r§eTesting end of string\n§e formatting character");
MessageTranslator.init();
}