RFR: 8337408: Use GetTempPath2 API instead of GetTempPath [v2]
Chris Plummer
cjplummer at openjdk.org
Thu Sep 26 16:20:38 UTC 2024
On Thu, 15 Aug 2024 20:28:28 GMT, Dhamoder Nalla <dhanalla at openjdk.org> wrote:
>> Use the GetTempPath2 APIs instead of the GetTempPath APIs in native code across the OpenJDK repository to retrieve the temporary directory path, as GetTempPath2 provides enhanced security. While GetTempPath may still function without errors, using GetTempPath2 reduces the risk of potential exploits for users.
>>
>>
>> The code to dynamically load GetTempPath2 is duplicated due to the following reasons. I would appreciate any suggestions to remove the duplication where possible:
>>
>> 1. The changes span across four different folders—java.base, jdk.package, jdk.attach, and hotspot—with no shared code between them.
>> 2. Some parts of the code use version A, while others use version W (ANSI vs. Unicode).
>> 3. Some parts of the code are written in C others in C++.
>
> Dhamoder Nalla has updated the pull request incrementally with one additional commit since the last revision:
>
> fix missing code
I don't have a suggestion for maintaining compatibility other than not making the change.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20600#issuecomment-2377406509
More information about the serviceability-dev
mailing list