Changeset 287575 in webkit

Timestamp:

Jan 4, 2022, 10:10:42 AM (4 years ago)

Author:

ysuzuki@apple.com

Message:

WebAssembly i32.atomic.wait timeout value incorrectly interpreted by factor 1000
​https://bugs.webkit.org/show_bug.cgi?id=234833

Reviewed by Michael Saboff.

Wasm atomics' timeout should be interpreted as nanoseconds.

• wasm/WasmOperations.cpp:

(JSC::Wasm::wait):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• wasm/WasmOperations.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2022-01-04  Yusuke Suzuki  <ysuzuki@apple.com>
+
+        WebAssembly i32.atomic.wait timeout value incorrectly interpreted by factor 1000
+        https://bugs.webkit.org/show_bug.cgi?id=234833
+
+        Reviewed by Michael Saboff.
+
+        Wasm atomics' timeout should be interpreted as nanoseconds.
+
+        * wasm/WasmOperations.cpp:
+        (JSC::Wasm::wait):
+
 2022-01-03  Yusuke Suzuki  <ysuzuki@apple.com>
 

trunk/Source/JavaScriptCore/wasm/WasmOperations.cpp

@@ -828 +828 @@
 {
     Seconds timeout = Seconds::infinity();
-    if (timeoutInNanoseconds >= 0) {
-        int64_t timeoutInMilliseconds = timeoutInNanoseconds / 1000;
-        timeout = Seconds::fromMilliseconds(timeoutInMilliseconds);
-    }
+    if (timeoutInNanoseconds >= 0)
+        timeout = Seconds::fromNanoseconds(timeoutInNanoseconds);
     bool didPassValidation = false;
     ParkingLot::ParkResult result;