CVE-2024-36116 (published 2024-06-19T18:15Z)

Reposilite is an open source, lightweight and easy-to-use repository manager for Maven based artifacts in JVM ecosystem. Reposilite provides support for JavaDocs files, which are archives that contain documentation for artifacts. Specifically, JavadocEndpoints.kt controller allows to expand the javadoc archive into the server's file system and return its content. The problem is in the way how the archives are expanded, specifically how the new filename is created. The `file.name` taken from the archive can contain path traversal characters, such as '/../../../anything.txt', so the resulting extraction path can be outside the target directory. If the archive is taken from an untrusted source, such as Maven Central or JitPack for example, an attacker can craft a special archive to overwrite any local file on Reposilite instance. This could lead to remote code execution, for example by placing a new plugin into the '$workspace$/plugins' directory. Alternatively, an attacker can overwrite the content of any other package. Note that the attacker can use its own malicious package from Maven Central to overwrite any other package on Reposilite. Reposilite has addressed this issue in version 3.5.12. Users are advised to upgrade. There are no known workarounds for this vulnerability. This issue was discovered and reported by the GitHub Security lab and is also tracked as GHSL-2024-073.


Weakness information:
cwe_id: NVD-CWE-noinfo
cwe_name: Insufficient Information
description: There is insufficient information about the issue to classify it; details are unkown or unspecified.
extended_description: Insufficient Information

Fix information:
repo_url: https://github.com/dzikoysk/reposilite
commit_details: repo_url: https://github.com/dzikoysk/reposilite
author: dzikoysk
author_date: 2024-05-03 17:22:40+02:00
msg: GHSA-frvj-cfq4-3228 Skip potentially malicious files in javadoc archives
repository: repo_url: https://github.com/dzikoysk/reposilite
repo_name: dzikoysk/reposilite
description: Lightweight and easy-to-use repository management software dedicated for the Maven based artifacts in the JVM ecosystem 📦
repo_language: Kotlin
owner: dzikoysk
file_changes: filename: JavadocContainerService.kt
change_type: ModificationType.MODIFY
diff: @@ -8,21 +8,22 @@ import com.reposilite.shared.badRequest
 import com.reposilite.shared.badRequestError
 import com.reposilite.shared.errorResponse
 import com.reposilite.shared.notFound
+import com.reposilite.status.FailureFacade
 import com.reposilite.storage.api.FileType.FILE
 import com.reposilite.storage.api.Location
+import com.reposilite.storage.api.toLocation
 import com.reposilite.storage.getSimpleName
 import com.reposilite.token.AccessTokenIdentifier
 import io.javalin.http.HttpStatus.INTERNAL_SERVER_ERROR
-import panda.std.Result
-import panda.std.asSuccess
 import java.io.FileOutputStream
 import java.io.InputStream
 import java.io.OutputStream
 import java.nio.file.Files
 import java.nio.file.Path
-import java.nio.file.Paths
 import java.util.jar.JarFile
 import kotlin.io.path.outputStream
+import panda.std.Result
+import panda.std.asSuccess
 
 private const val INDEX_FILE = "index.html"
 
@@ -33,6 +34,7 @@ internal class JavadocContainer(
 )
 
 internal class JavadocContainerService(
+    private val failureFacade: FailureFacade,
     private val mavenFacade: MavenFacade,
     private val javadocFolder: Path
 ) {
@@ -116,7 +118,6 @@ internal class JavadocContainerService(
             !jarPath.getSimpleName().contains("javadoc.jar") -> badRequestError("Invalid javadoc jar! Name must contain: 'javadoc.jar'")
             Files.isDirectory(jarPath) -> badRequestError("JavaDoc jar path has to be a file!")
             !Files.isDirectory(javadocUnpackPath) -> badRequestError("Destination must be a directory!")
-
             else -> jarPath.toAbsolutePath().toString()
                 .let { JarFile(it) }
                 .use { jarFile ->
@@ -124,16 +125,32 @@ internal class JavadocContainerService(
                         return errorResponse(INTERNAL_SERVER_ERROR, "Invalid javadoc.jar given for extraction")
                     }
 
-                    jarFile.entries().asSequence().forEach { file ->
-                        if (file.isDirectory) {
-                            return@forEach
+                    jarFile
+                        .entries()
+                        .asSequence()
+                        .forEach { file ->
+                            if (file.isDirectory) {
+                                return@forEach
+                            }
+
+                            // GHSA-frvj-cfq4-3228: treat archive file name as external path that can be malicious
+                            val processedArchiveFileLocation = file.name.toLocation()
+
+                            processedArchiveFileLocation
+                                .toPath()
+                                .map { javadocUnpackPath.resolve(it) }
+                                .peek {
+                                    it.parent?.also { parent -> Files.createDirectories(parent) }
+                                    jarFile.getInputStream(file).copyToAndClose(it.outputStream())
+                                }
+                                .onError {
+                                    failureFacade.throwException(
+                                        "Malicious resource path detected: $processedArchiveFileLocation in $jarPath",
+                                        IllegalArgumentException("Malicious resource path detected: $it")
+                                    )
+                                }
                         }
-
-                        val path = Paths.get(javadocUnpackPath.toString() + "/" + file.name)
-
-                        path.parent?.also { parent -> Files.createDirectories(parent) }
-                        jarFile.getInputStream(file).copyToAndClose(path.outputStream())
-                    }.asSuccess<Unit, ErrorResponse>()
+                        .asSuccess<Unit, ErrorResponse>()
                 }
                 .also { Files.deleteIfExists(jarPath) }
         }

programming_language: Swift