Filename buffers too small after fix for 6938627

[Andreas Kohn]

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.


Attached patch changes the buffers to PATH_MAX+1 bytes, the same value
used by other places that call get_temp_directory().

Regards,
--
Andreas

-- 
Never attribute to malice that which can be adequately explained by
stupidity.                                        -- Hanlon's Razor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6938627-bufs.diff
Type: text/x-patch
Size: 1586 bytes
Desc: 
Url : http://mail.openjdk.java.net/pipermail/hotspot-dev/attachments/20100411/7bc6aa9b/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
Url : http://mail.openjdk.java.net/pipermail/hotspot-dev/attachments/20100411/7bc6aa9b/attachment-0001.bin