stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8010182: Thread safety of Thread get/setName()

Browse Source 
Reviewed-by: dholmes, alanb, mduigou
This commit is contained in:
Chris Hegarty 2013-05-22 13:50:53 +01:00
parent eb6c8c5bc8
commit 64b3efd73b
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
jdk/src/share/classes/java/lang/Thread.java
View File

@ -145,10 +145,10 @@ class Thread implements Runnable {
registerNatives(); registerNatives();
} }
private char name[]; private volatile char name[];
private int priority; private int priority;
private Thread threadQ; private Thread threadQ;
private long eetop; private long eetop;
/* Whether or not to single_step this thread. */ /* Whether or not to single_step this thread. */
private boolean single_step; private boolean single_step;
@ -1135,7 +1135,7 @@ class Thread implements Runnable {
* @see #getName * @see #getName
* @see #checkAccess() * @see #checkAccess()
*/ */
public final void setName(String name) { public final synchronized void setName(String name) {
checkAccess(); checkAccess();
this.name = name.toCharArray(); this.name = name.toCharArray();
if (threadStatus != 0) { if (threadStatus != 0) {
@ -1150,7 +1150,7 @@ class Thread implements Runnable {
* @see #setName(String) * @see #setName(String)
*/ */
public final String getName() { public final String getName() {
return String.valueOf(name); return new String(name, true);
} }
/** /**