RFR: 8303070: Memory leak in DCmdArgument<char*>::parse_value [v4]

[Johan Sjölen]

On Sat, 25 Feb 2023 02:36:36 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> If we have a default value for the char* DCmdArgument we copy it into the `_value` field using `parse_value` to make a copy in C-heap. If we then parse an actual argument value, we replace the default but don't free it. The parse method needs to use realloc.
>> 
>> Thanks to @jcking for spotting the cause.
>> 
>> Testing: tiers 1-3 
>> 
>> Thanks.
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update test coverage for the empty and null cases

LGTM, thanks.

-------------

Marked as reviewed by jsjolen (Committer).

PR: https://git.openjdk.org/jdk/pull/12737