RFR: 8271003: hs_err improvement: handle CLASSPATH env setting longer than O_BUFLEN [v2]
dholmes at openjdk.java.net
Tue Aug 3 22:05:33 UTC 2021
On Tue, 3 Aug 2021 16:14:57 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Please review this small enhancement for addressing the problem of the CLASSPATH env variable setting being truncated in a hs err log.
>> For printing a char string, it doesn't need to go through `do_vsnprintf()` which does the truncation based on the input buffer length. The change is local to the code path pertaining to hs err log.
>> - [x] tiers 1, 2 (including the new test)
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> @tstuefe comments
One comment request.
src/hotspot/share/runtime/os.cpp line 992:
> 990: st->print("%s", env_list[i]);
> 991: st->print("=");
> 992: st->print("%s", envvar);
I suggest adding a comment like:
// Use separate cr() printing to avoid unnecessary buffer operations that might cause truncation
Marked as reviewed by dholmes (Reviewer).
More information about the hotspot-runtime-dev