QV4: Support printing arrays with circular references

Fixes: QTBUG-81105
Change-Id: Iaf0597cea3a5572f020c5f87a843774f33cc01fd
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

1 files changed, 9 insertions, 4 deletions

diff --git a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
index d634a48443..f5e723419e 100644
--- a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
+++ b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
@@ -1553,11 +1553,12 @@ static QString jsStack(QV4::ExecutionEngine *engine) {
     return stack;
 }
 
-static QString serializeArray(Object *array, ExecutionEngine *v4) {
+static QString serializeArray(Object *array, ExecutionEngine *v4, QSet<QV4::Heap::Object *> &alreadySeen) {
     Scope scope(v4);
     ScopedValue val(scope);
     QString result;
 
+    alreadySeen.insert(array->d());
     result += QLatin1Char('[');
     const uint length = array->getLength();
     for (uint i = 0; i < length; ++i) {
@@ -1565,12 +1566,15 @@ static QString serializeArray(Object *array, ExecutionEngine *v4) {
             result += QLatin1Char(',');
         val = array->get(i);
         if (val->isManaged() && val->managed()->isArrayLike())
-            result += serializeArray(val->objectValue(), v4);
+            if (!alreadySeen.contains(val->objectValue()->d()))
+                result += serializeArray(val->objectValue(), v4, alreadySeen);
+            else
+                result += QLatin1String("[Circular]");
         else
             result += val->toQStringNoThrow();
     }
     result += QLatin1Char(']');
-
+    alreadySeen.remove(array->d());
     return result;
 };
 
@@ -1600,8 +1604,9 @@ static ReturnedValue writeToConsole(const FunctionObject *b, const Value *, cons
         if (i != start)
             result.append(QLatin1Char(' '));
 
+        QSet<QV4::Heap::Object *> alreadySeenElements;
         if (argv[i].isManaged() && argv[i].managed()->isArrayLike())
-            result.append(serializeArray(argv[i].objectValue(), v4));
+            result.append(serializeArray(argv[i].objectValue(), v4, alreadySeenElements));
         else
             result.append(argv[i].toQStringNoThrow());
     }