JDK-8228402 : chdir() and write() errors are not handled in jpackage
  • Type: Bug
  • Component: tools
  • Sub-Component: jpackage
  • Affected Version: internal
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2019-07-18
  • Updated: 2019-07-25
  • Resolved: 2019-07-25
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
internalFixed
Related Reports
Blocks :  
JDK-8222778 - Packaging Tool (JEP 343) on Linux/AArch64
Relates :  
JDK-8212780 - Packaging Tool Implementation
Relates :  
JDK-8226904 - current working directory wrong running jpackage app
Description
In libapplauncher chdir() is called in LinuxPlatform::SetCurrentDirectory() and PosixPlatform::SetCurrentDirectory(), write() is called in PosixProcess::SetInput(). Return code is ignored currently.
Launcher should exit with an error if such calls were not successful.
Comments
webrev: http://cr.openjdk.java.net/~herrick/8228402/
23-07-2019
1.) Since JDK-8226904, the code now no longer changes the current directory, however each platform still has the SetCurrentDirectory() function. All these can be removed. 2.) The return value from write() in PosixProcess::SetInput() can be handled by asserting that return value is >= 0
23-07-2019