RFR: JDK-8308350: Increase buffer size for jspawnhelper arguments

Thomas Stuefe stuefe at openjdk.org
Thu May 18 07:22:23 UTC 2023


Trivial fix for a small problem.

jspawnhelper gets handed several file descriptors as arguments. The buffer size for this string is too small (7 chars per fd) to print out every conceivable int. This will overun the buffer if we happen to have fds larger than (printed size) 7 characters. This could lead to crashes or malfunctions if the parent VM has opened a large amount of file descriptors.

Note that on Linux, this can normally not happen since the kernel limits the number of open file descriptors per process to 1M, and these fds are still printable within the limits of this buffer. It is possible to get more fds per process, but only via kernel patch. But we still should not rely on that. And there is also still MacOS using the same mechanism.

-------------

Commit messages:
 - JDK-8308350-Increase-buffer-size-for-jspawnhelper-arguments

Changes: https://git.openjdk.org/jdk/pull/14045/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14045&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8308350
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/14045.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14045/head:pull/14045

PR: https://git.openjdk.org/jdk/pull/14045


More information about the core-libs-dev mailing list