Filename buffers too small after fix for 6938627

[David Holmes]

Andreas,

I've filed 6944822 in our internal bug system for this.

Regards,
David Holmes

Andreas Kohn said the following on 04/18/10 18:26:
> On Sun, 2010-04-11 at 20:57 +0200, Andreas Kohn wrote:
>> Hi,
>>
>> while reading a bit the hotspot sources I noticed a potential issue with
>> the fix for 6938627 (Make temporary directory use property
>> java.io.tmpdir when specified) in some places.
>>
>> Before the fix the callers of get_temp_directory() could hardcode the
>> size of the filename buffer to a small number, but now that
>> get_temp_directory() returns a value settable by the user this looks a
>> bit dangerous.
>>
>> In particular:
>> attachListener_linux.cpp:AttachListener::is_init_trigger()
>> attachListener_solaris.cpp:AttachListener::is_init_trigger()
>>   both use a 128 byte buffer
>>
>> os_linux.cpp:linux_wrap_code()
>>   uses a 40 byte buffer.
> 
> 
> Should I open a bug for this one? 
> 
> Regards,
> --
> Andreas
> 
>