[JSC] Use tryMakeString for error message on invalid destructuring

https://bugs.webkit.org/show_bug.cgi?id=300190

Reviewed by Keith Miller.

This patch changes to use `tryMakeString` for error message on invalid
destructuring for avoiding crash.

* Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::emitRequireObjectCoercibleForDestructuring):

Canonical link: https://commits.webkit.org/301053@main

Author: sosukesuzuki

Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp

@@ -4994,9 +4994,13 @@ void BytecodeGenerator::emitRequireObjectCoercibleForDestructuring(RegisterID* v
     Ref<Label> target = newLabel();
     OpJnundefinedOrNull::emit(this, value, target->bind(this));
 
-    if (propertyName && !propertyName->isNull())
-        emitThrowTypeError(Identifier::fromString(m_vm, makeString("Cannot destructure property '"_s, propertyName->string(), "' from null or undefined value"_s)));
-    else
+    if (propertyName && !propertyName->isNull()) {
+        auto errorMessageStr = tryMakeString("Cannot destructure property '"_s, propertyName->string(), "' from null or undefined value"_s);
+        if (!errorMessageStr) [[unlikely]]
+            emitThrowTypeError("Cannot destructure null or undefined value"_s);
+        else
+            emitThrowTypeError(Identifier::fromString(m_vm, errorMessageStr));
+    } else
         emitThrowTypeError("Cannot destructure null or undefined value"_s);
 
     emitLabel(target.get());