Request for review: 7141739: [osx] Local attach fails if java.io.tmpdir is set (original) (raw)
Daniel D. Daugherty daniel.daugherty at oracle.com
Wed Feb 1 14:59:32 PST 2012
- Previous message: Request for review: 7141739: [osx] Local attach fails if java.io.tmpdir is set
- Next message: Request for review: 7141739: [osx] Local attach fails if java.io.tmpdir is set
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2/1/12 9:05 AM, Staffan Larsen wrote:
Please review the following change.
webrev: http://cr.openjdk.java.net/~sla/7141739/webrev.00/ bug: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7141739 This is a followup of bug 7132199 which is fixed in jdk8 and jdk7u-dev. If the user sets -Djava.io.tmpdir to something else than /tmp, the attach framework will fail (for example jstack). The reason is that Hotspot and the jdk does not agree on which directory the well-known .javapid and .attachpid files should be in. On OSX Hotspot will always create these in the per-user secure temporary directory, but the jdk will look in java.io.tmpdir. This fix makes sure the jdk looks in the same location as Hotspot. Thanks, /Staffan
Please update copyright years to 2012.
src/solaris/classes/sun/tools/attach/BsdVirtualMachine.java line 73: Always creates the ".attach_pid" file in tmpdir. The HSX side matches this and always looks for the ".attach_pid" file in tmpdir. Perhaps we should change Linux and Solaris to always create and look for the ".attach_pid" file in tmpdir...
src/solaris/native/sun/tools/attach/BsdVirtualMachine.c No comments.
- Previous message: Request for review: 7141739: [osx] Local attach fails if java.io.tmpdir is set
- Next message: Request for review: 7141739: [osx] Local attach fails if java.io.tmpdir is set
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]