GitHub Advisoryadvisory · Maven

GHSA-269q-hmxg-m83q Local Information Disclosure Vulnerability in io.netty:netty-codec-http

MEDIUM Published 2022-05-10 Modified 2022-05-10 Official page ↗
Aliases
CVE-2022-24823
Ecosystem
Maven
Packages
io.netty:netty-codec-http
Severity
Moderate
CVSS vector
CVSS:3.1/AV:L/AC:L/PR:L/UI:N/S:U/C:H/I:N/A:N
Weaknesses
CWE-378, CWE-379, CWE-668
Reviewed
2022-05-10

Affected packages

• Maven io.netty:netty-codec-http: < 4.1.77.Final → fixed in 4.1.77.Final

Description

Description ### GHSA-5mcr-gq6c-3hq2 (CVE-2021-21290) contains an insufficient fix for the vulnerability identified. Impact ### When netty's multipart decoders are used local information disclosure can occur via the local system temporary directory if temporary storing uploads on the disk is enabled. This only impacts applications running on Java version 6 and lower. Additionally, this vulnerability impacts code running on Unix-like systems, and very old versions of Mac OSX and Windows as they all share the system temporary directory between all users. Vulnerability Details ### To fix the vulnerability the code was changed to the following: ```java @SuppressJava6Requirement(reason = "Guarded by version check") public static File createTempFile(String prefix, String suffix, File directory) throws IOException { if (javaVersion() >= 7) { if (directory == null) { return Files.createTempFile(prefix, suffix).toFile(); } return Files.createTempFile(directory.toPath(), prefix, suffix).toFile(); } if (directory == null) { return File.createTempFile(prefix, suffix); } File file = File.createTempFile(prefix, suffix, directory); // Try to adjust the perms, if this fails there is not much else we can do... file.setReadable(false, false); file.setReadable(true, true); return file; } ``` Unfortunately, this logic path was left vulnerable: ```java if (directory == null) { return File.createTempFile(prefix, suffix); } ``` This file is still readable by all local users. Patches ### Update to 4.1.77.Final Workarounds ### Specify your own `java.io.tmpdir` when you start the JVM or use `DefaultHttpDataFactory.setBaseDir(...)` to set the directory to something that is only readable by the current user or update to Java 7 or above. References ### • CWE-378: Creation of Temporary File With Insecure Permissions • CWE-379: Creation of Temporary File in Directory with Insecure Permissions For more information ### If you have any questions or comments about this advisory: Open an issue in netty

Related

Alias · CVE (1)

  • CVE-2022-24823 Netty is an open-source, asynchronous event-driven network application framework. vulnerability

Weakness · CWE (3)

  • CWE-378 Creation of Temporary File With Insecure Permissions weakness
  • CWE-379 Creation of Temporary File in Directory with Insecure Permissions weakness
  • CWE-668 Exposure of Resource to Wrong Sphere weakness

References