RFR(M): 8038201: Clean up misleading usage of malloc() in init_system_properties_values()

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Mar 25 05:13:45 UTC 2014

Hi Goetz,

Do you really have to change /**/ comments to //?
Without that you would get much clear visible changes.

Changes looks fine to me. I wish we could do this in os_posix.cpp but it is for an other time.


On 3/24/14 2:50 PM, Lindenmaier, Goetz wrote:
> Hi,
> please review and test this change. I please need a sponsor.
> http://cr.openjdk.java.net/~goetz/webrevs/8038201-sec/webrev.00/
> This change addresses the implementation of init_system_properties_values
> on aix, linux, solaris and bsd.
> In init_system_properties_values a macro was defined mapping malloc to
> NEW_C_HEAP_ARRAY. NEW_C_HEAP_ARRAY checks for successful allocation
> or exits the VM.  The code of this method handles the allocated pointers as
> allocated with real malloc, i.e.,  there were checks for NULL and calls to free().
> This change replaces the macro malloc with NEW_C_HEAP_ARRAY and removes
> the unnecessary checks making the code clearer.  Also, it uses a local array where
> possible.
> The allocated memory is passed to calls that end up at SystemProperty::set_value().
> set_value copies the strings passed.  Thus the memory allocated in
> init_system_properties_values must be freed after these calls.  Most of these
> frees were missing.
> This change adds the missing frees.
> Testing this change I ran into a warning in ppc code which I fixed, too.
> I did some local test.  I'll get broader tests by Wednesday.
> Best regards,
>    Goetz.

More information about the hotspot-dev mailing list