MrNerdy42 / keywizard-legacy

A simple Minecraft mod that makes finding and fixing key conflicts easier.
https://www.curseforge.com/minecraft/mc-mods/keyboard-wizard
Other
5 stars 3 forks source link

regex error #5

Closed The28AWG closed 5 years ago

The28AWG commented 6 years ago

Hello! You find pattern dont safe. https://github.com/MrNerdy42/Keyboard-Wizard/blob/4b68ebc939861fb563f4bd1b4293b5face2763ab/src/main/java/com/github/mrnerdy42/keywizard/gui/GuiBindingList.java#L88 https://github.com/MrNerdy42/Keyboard-Wizard/blob/4b68ebc939861fb563f4bd1b4293b5face2763ab/src/main/java/com/github/mrnerdy42/keywizard/gui/GuiBindingList.java#L124 I sended [ from text input and client crashed.

[15:24:52] [main/FATAL]: Reported exception thrown! net.minecraft.util.ReportedException: Ticking screen at net.minecraft.client.Minecraft.func_71407_l(Minecraft.java:1790) ~[bib.class:?] at net.minecraft.client.Minecraft.func_71411_J(Minecraft.java:1096) ~[bib.class:?] at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:397) [bib.class:?] at net.minecraft.client.main.Main.main(SourceFile:123) [Main.class:?] at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_151] at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_151] at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_151] at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_151] at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.12.jar:?] at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?] Caused by: java.util.regex.PatternSyntaxException: Unclosed character class near index 8 (?i).[. ^ at java.util.regex.Pattern.error(Pattern.java:1955) ~[?:1.8.0_151] at java.util.regex.Pattern.clazz(Pattern.java:2548) ~[?:1.8.0_151] at java.util.regex.Pattern.sequence(Pattern.java:2063) ~[?:1.8.0_151] at java.util.regex.Pattern.expr(Pattern.java:1996) ~[?:1.8.0_151] at java.util.regex.Pattern.compile(Pattern.java:1696) ~[?:1.8.0_151] at java.util.regex.Pattern.(Pattern.java:1351) ~[?:1.8.0_151] at java.util.regex.Pattern.compile(Pattern.java:1028) ~[?:1.8.0_151] at java.util.regex.Pattern.matches(Pattern.java:1133) ~[?:1.8.0_151] at java.lang.String.matches(String.java:2121) ~[?:1.8.0_151] at com.github.mrnerdy42.keywizard.gui.GuiBindingList.lambda$getBindings$6(GuiBindingList.java:124) ~[GuiBindingList.class:?] at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:174) ~[?:1.8.0_151] at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948) ~[?:1.8.0_151] at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481) ~[?:1.8.0_151] at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471) ~[?:1.8.0_151] at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:545) ~[?:1.8.0_151] at java.util.stream.AbstractPipeline.evaluateToArrayNode(AbstractPipeline.java:260) ~[?:1.8.0_151] at java.util.stream.ReferencePipeline.toArray(ReferencePipeline.java:438) ~[?:1.8.0_151] at com.github.mrnerdy42.keywizard.gui.GuiBindingList.getBindings(GuiBindingList.java:124) ~[GuiBindingList.class:?] at com.github.mrnerdy42.keywizard.gui.GuiBindingList.updateList(GuiBindingList.java:95) ~[GuiBindingList.class:?] at com.github.mrnerdy42.keywizard.gui.GuiKeyWizard.func_73876_c(GuiKeyWizard.java:402) ~[GuiKeyWizard.class:?] at net.minecraft.client.Minecraft.func_71407_l(Minecraft.java:1777) ~[bib.class:?]