clientcommands icon indicating copy to clipboard operation
clientcommands copied to clipboard

Published 20 hours ago • verified publisher icon Earthcomputer

`/ccalc --parse` can stack overflow due to excessive recursion displaying the chat output

Open James103 opened this issue 4 years ago • 2 comments

To reproduce,

  1. Type /ccalc --parse (with the trailing space).
  2. Copy the following string (select the below text and hit CTRL+C) or click the copy to clipboard button that appears when you hover over the below text):
1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+1+
  1. Paste the string into chat about 10 times.
  2. Type a final 1, so the command looks like /ccalc --parse 1+1+1+1+1+...+1+1+1+1 where there are at least 5,000 consecutive occurrences of 1+.
  3. Hit enter.
  4. The game crashes.

Crash report: crash-2021-10-06_22.20.08-client.txt Note: The exception stack trace is shortened, but it repeats the following two lines hundreds of times:

	at net.minecraft.text.TranslatableText.visitSelf(TranslatableText:146)
	at net.minecraft.text.Text.visit(Text:96)

Note 2: This does not happen without the --parse flag.

James103 avatar Oct 07 '21 03:10 James103

I think I implemented a fix for this at some point, mustn’t’ve fixed it I guess - strange.

xpple avatar Oct 07 '21 14:10 xpple

Can't reproduce in Client Commands version 2.6.6 due to the vanilla chat message limit of 254 characters.

James103 avatar Apr 19 '22 12:04 James103