RFR(M): 8038201: Clean up misleading usage of malloc() in init_system_properties_values()
coleen.phillimore at oracle.com
Tue Mar 25 13:46:28 UTC 2014
If os::init_system_properties() is really the same for all of these
platforms, I would like to see it moved to os_posix.cpp. It doesn't
seem to make sense to change it the same way 4 times. Jerry Thornbrough
is working on the RFE to move "posix" into "xnix", and make more code
common but this could precede that.
On 3/25/14 1:13 AM, Vladimir Kozlov wrote:
> 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:
>> please review and test this change. I please need a sponsor.
>> This change addresses the implementation of
>> 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
>> The allocated memory is passed to calls that end up at
>> 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,
More information about the hotspot-dev