|
+1
Thanks,
Serguei
On 12/3/18 12:09, Chris Plummer wrote:
...and I've been waiting for the
webrev update to give it a thumbs up.
Chris
On 11/30/18 1:11 AM, gary.ad...@oracle.com
wrote:
I have not refreshed the
webrev, yet.
Waiting to see if there are any additional comments
before updating the webrev.
The spaces and the assignments you noticed
have been fixed.
On 11/29/18 5:27 PM, JC Beyler wrote:
Hi Gary,
Somehow I still see the same webrev? Has it been
updated and my browser does not want to update it?
Thanks!
Jc
Sorry, coding before coffee this morning.
Fixed.
On 11/29/18, 11:54 AM, JC Beyler wrote:
Hi Gary,
f = f.getCanonicalFile()?
(Nit, there seems to be a space missing
before the =).
In the other two files, you just call the
method but don't update the variable f
making the call useless, no? (The patch in
the bug has the assigments for all the
cases),
Jc
If a process
exits during an attempt to attach to it, the
.attach_pid file
will not be removed properly, if the path used
included symbolic link
traversal,
which is typically done for
"/proc/<pid>/cwd/". Using
getCanonicalFile()
before the
initial file is created should prevent this
edge case for left over files.
Webrev: http://cr.openjdk.java.net/~gadams/8214300/webrev.00/index.html
Issue: https://bugs.openjdk.java.net/browse/JDK-8214300
Testing is in progress for every attach test I
can find, but there is no
explicit test available for the case where the
attachee exits while the
attacher
is making the initial connection.
--
--
|
