Fix a data race found by tsan in Thread.java, interrupted should be accessed behind a lock. This also has the nice side effect of making the wasInterrupted changes in NSObject+JavaObject atomic.

PiperOrigin-RevId: 748771858

Author: j2objc-copybara

jre_emul/Classes/NSObject+JavaObject.m

@@ -123,14 +123,14 @@ static void doWait(id obj, long long timeout) {
   }
   JavaLangThread *javaThread = JavaLangThread_currentThread();
   int result = OBJC_SYNC_SUCCESS;
-  if (!javaThread->interrupted_) {
+  if (![javaThread isInterrupted]) {
     assert(javaThread->blocker_ == nil);
     javaThread->blocker_ = obj;
     result = objc_sync_wait(obj, timeout);
     javaThread->blocker_ = nil;
   }
-  jboolean wasInterrupted = javaThread->interrupted_;
-  javaThread->interrupted_ = false;
+  // Check if the thread was interrupted after the wait and also clears the interrupted bit.
+  jboolean wasInterrupted = [JavaLangThread interrupted];
   if (wasInterrupted) {
     @throw AUTORELEASE([[JavaLangInterruptedException alloc] init]);
   }

jre_emul/Classes/java/lang/Thread.java

@@ -65,7 +65,7 @@ public class Thread implements Runnable {
   private int priority = NORM_PRIORITY;
   private volatile UncaughtExceptionHandler uncaughtExceptionHandler;
   private boolean isDaemon;
-  boolean interrupted;
+  private boolean interrupted;
   private ClassLoader contextClassLoader;
   ThreadLocal.ThreadLocalMap threadLocals = null;
   ThreadLocal.ThreadLocalMap inheritableThreadLocals = null;
@@ -688,7 +688,9 @@ public static native boolean interrupted() /*-[
    * @see Thread#interrupted
    */
   public boolean isInterrupted() {
-    return interrupted;
+    synchronized (nativeThread) {
+      return interrupted;
+    }
   }
 
   /**