Syncronize and check that they are not trying to break us

urielsalis · May 11, 2021 · f64f680 · f64f680
1 parent 9d6fb46
commit f64f680
Showing 1 changed file with 15 additions and 6 deletions.
diff --git a/src/main/kotlin/me/urielsalis/mccrashlib/deobfuscator/Deobfuscator.kt b/src/main/kotlin/me/urielsalis/mccrashlib/deobfuscator/Deobfuscator.kt
@@ -18,17 +18,26 @@ fun getDeobfuscation(modded: Boolean, version: String, content: String, isClient
     if (modded) {
         return null
     }
+
+    if (version.isBlank() || version.contains("\\") || version.contains("/")) {
+        return null
+    }
+
     val name = if (isClient) {
         "$version-client"
     } else {
         "$version-server"
     }
-    val mappingFile = File(mappingsFile, name)
-    if (!mappingFile.exists()) {
-        downloadMapping(version, name, isClient)
-    }
-    if (!mappingFile.exists()) {
-        return null
+
+    val mappingFile = synchronized(mappingsFile) {
+        val mappingFile = File(mappingsFile, name)
+        if (!mappingFile.exists()) {
+            downloadMapping(version, name, isClient)
+        }
+        if (!mappingFile.exists()) {
+            return null
+        }
+        mappingFile
     }
     val retrace = ReTrace(ReTrace.REGULAR_EXPRESSION, false, true, mappingFile)
     val stringWriter = StringWriter()