On Wed, 29 Jan 2025 20:42:30 GMT, Naoto Sato wrote:
> Force opening "tzmappings" file in text mode. Confirmed the fix with customer
> provided test case. Also replaced the file open function with the safer one.
Thanks for the reviews!
-
PR Comment: https://git.openjdk.org/jdk/pull
On Wed, 29 Jan 2025 20:42:30 GMT, Naoto Sato wrote:
> Force opening "tzmappings" file in text mode. Confirmed the fix with customer
> provided test case. Also replaced the file open function with the safer one.
Marked as reviewed by joehw (Reviewer).
-
PR Review: https://git.openj
On Wed, 29 Jan 2025 20:42:30 GMT, Naoto Sato wrote:
> Force opening "tzmappings" file in text mode. Confirmed the fix with customer
> provided test case. Also replaced the file open function with the safer one.
Fix looks correct to force "text mode". Switch to `fopen_s` also looks good.
--
Force opening "tzmappings" file in text mode. Confirmed the fix with customer
provided test case. Also replaced the file open function with the safer one.
-
Commit messages:
- initial commit
Changes: https://git.openjdk.org/jdk/pull/23356/files
Webrev: https://webrevs.openjdk.org
