Please review 6938627: Make temporary directory use property when specified

David Schlosnagle schlosna at
Fri Mar 26 17:07:38 PDT 2010


There seems to be a couple issues on line 469 of src/os/linux/vm/attachListener_linux.cpp:
- It seems like there should be a slash in the format string to match the Solaris version.
- The call to os::get_temp_directory() appears to be missing the 'y' at the end.

469     sprintf(fn, "%s.attach_pid%d", os::get_temp_director(),
                       ^                                   ^^
470                                    os::current_process_id());

600     sprintf(fn, "%s/.attach_pid%d", os::get_temp_directory(),
                       ^                                     ^^
601                                     os::current_process_id());


On Mar 26, 2010, at 6:11 PM, Coleen Phillimore wrote:

> Summary: Get property in os::get_temp_directory() and call this instead of harcoding "/tmp".  Don't assume trailing file_separator either.
> open webrev at
> bug link at
> Thanks,
> Coleen

More information about the hotspot-dev mailing list