Add more exception checking

Author: sosukesuzuki

Source/JavaScriptCore/runtime/JSMicrotask.cpp

@@ -409,12 +409,13 @@ void runInternalMicrotask(JSGlobalObject* globalObject, InternalMicrotask task,
             if (auto* promise = jsDynamicCast<JSPromise*>(promiseOrCapability)) {
                 promise->rejectPromise(globalObject, error);
 #if USE(BUN_JSC_ADDITIONS)
-                // Restore async context after promise operation
+                // Restore async context before exception check (must restore even if exception occurs)
                 if (hasAsyncContext) {
                     if (auto* asyncContextData = globalObject->m_asyncContextData.get())
                         asyncContextData->putInternalField(vm, 0, previousAsyncContext);
                 }
 #endif
+                RETURN_IF_EXCEPTION(scope, void());
                 return;
             }
 
@@ -439,12 +440,13 @@ void runInternalMicrotask(JSGlobalObject* globalObject, InternalMicrotask task,
         if (auto* promise = jsDynamicCast<JSPromise*>(promiseOrCapability)) {
             promise->resolvePromise(globalObject, result);
 #if USE(BUN_JSC_ADDITIONS)
-            // Restore async context after promise operation
+            // Restore async context before exception check (must restore even if exception occurs)
             if (hasAsyncContext) {
                 if (auto* asyncContextData = globalObject->m_asyncContextData.get())
                     asyncContextData->putInternalField(vm, 0, previousAsyncContext);
             }
 #endif
+            RETURN_IF_EXCEPTION(scope, void());
             return;
         }
 

Source/JavaScriptCore/runtime/JSPromise.cpp

@@ -424,6 +424,7 @@ void JSPromise::rejectPromise(JSGlobalObject* globalObject, JSValue argument)
 void JSPromise::fulfillPromise(JSGlobalObject* globalObject, JSValue argument)
 {
     VM& vm = globalObject->vm();
+    auto scope = DECLARE_THROW_SCOPE(vm);
 
     ASSERT(status() == Status::Pending);
     uint32_t flags = this->flags();
@@ -432,7 +433,7 @@ void JSPromise::fulfillPromise(JSGlobalObject* globalObject, JSValue argument)
     internalField(Field::ReactionsOrResult).set(vm, this, argument);
     if (!reactions)
         return;
-    triggerPromiseReactions(globalObject, Status::Fulfilled, reactions, argument);
+    RELEASE_AND_RETURN(scope, triggerPromiseReactions(globalObject, Status::Fulfilled, reactions, argument));
 }
 
 void JSPromise::resolvePromise(JSGlobalObject* globalObject, JSValue resolution)