RFR: JDK-8280004: DCmdArgument<jlong>::parse_value() should handle NULL input [v3]
Thomas Stuefe
stuefe at openjdk.java.net
Fri Jan 28 08:13:53 UTC 2022
> Hi,
>
> may I have eyes please on this simple fix (Sonarcloud-inspired) which sanitizes the error output of DCmdArgument<jlong>::parse_value():
> - since the input can be NULL, we should handle NULL
> - we also limit parroting of invalid input to 64 chars max
> - we also do not need a temporary RA buffer to assemble the output
>
> Example output with patch, first a short, then a long invalid numeric parameter:
>
>
> thomas at starfish:~$ jcmd Simple GC.class_histogram -parallel=hallo
> 399746:
> java.lang.IllegalArgumentException: Integer parsing error in command argument '-parallel'. Could not parse: "hallo".
> thomas at starfish:~$ jcmd Simple GC.class_histogram -parallel=hallo012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789
> 399746:
> java.lang.IllegalArgumentException: Integer parsing error in command argument '-parallel'. Could not parse: "hallo01234567890123456789012345678901234567890123456789012345678...".
>
>
> Thanks, Thomas
Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
fix copyright
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7079/files
- new: https://git.openjdk.java.net/jdk/pull/7079/files/8af94135..a6ca495c
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7079&range=02
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7079&range=01-02
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/7079.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7079/head:pull/7079
PR: https://git.openjdk.java.net/jdk/pull/7079
More information about the hotspot-runtime-dev
mailing list