[MERGE #5897 @rajatd] Wrap NewScObjectNoCtor with safe reentrancy macros when called from reentracy safe path. OS#19873760

Merge pull request #5897 from rajatd:reentrancy-check-fix

Author: rajatd

lib/Runtime/Library/JavascriptFunction.cpp

@@ -866,7 +866,11 @@ using namespace Js;
         }
         else
         {
-            resultObject = JavascriptOperators::NewScObjectNoCtor(v, scriptContext);
+            BEGIN_SAFE_REENTRANT_CALL(scriptContext->GetThreadContext())
+            {
+                resultObject = JavascriptOperators::NewScObjectNoCtor(v, scriptContext);
+            }
+            END_SAFE_REENTRANT_CALL
         }
 
         // JavascriptOperators::NewScObjectNoCtor should have thrown if 'v' is not a constructor

lib/Runtime/Library/JavascriptProxy.cpp

@@ -2143,7 +2143,11 @@ namespace Js
                 // args.Values[0] will be null in the case where NewTarget is initially provided by proxy.
                 if (!isCtorSuperCall || !args.Values[0])
                 {
-                    newThisObject = JavascriptOperators::NewScObjectNoCtor(targetObj, scriptContext);
+                    BEGIN_SAFE_REENTRANT_CALL(scriptContext->GetThreadContext())
+                    {
+                        newThisObject = JavascriptOperators::NewScObjectNoCtor(targetObj, scriptContext);
+                    }
+                    END_SAFE_REENTRANT_CALL
                     args.Values[0] = newThisObject;
                 }
                 else