Memory leak in QScriptEngineAgent.

The documentation of the agent constructor specify that the agant is
owned by the engine. Even if the agent is not set to the engine

Reviewed-by: Kent Hansen

2 files changed, 1 insertions, 3 deletions

diff --git a/src/script/api/qscriptengine.cpp b/src/script/api/qscriptengine.cpp
index 033d92e..f4e9594 100644
--- a/src/script/api/qscriptengine.cpp
+++ b/src/script/api/qscriptengine.cpp
@@ -3585,9 +3585,6 @@ void QScriptEngine::setAgent(QScriptEngineAgent *agent)
         QScriptEngineAgentPrivate::get(d->activeAgent)->detach();
     d->activeAgent = agent;
     if (agent) {
-        int index = d->ownedAgents.indexOf(agent);
-        if (index == -1)
-            d->ownedAgents.append(agent);
         QScriptEngineAgentPrivate::get(agent)->attach();
     }
 }
diff --git a/src/script/api/qscriptengineagent.cpp b/src/script/api/qscriptengineagent.cpp
index b18f29d..193ee21 100644
--- a/src/script/api/qscriptengineagent.cpp
+++ b/src/script/api/qscriptengineagent.cpp
@@ -213,6 +213,7 @@ QScriptEngineAgent::QScriptEngineAgent(QScriptEngine *engine)
 {
     d_ptr->q_ptr = this;
     d_ptr->engine = QScriptEnginePrivate::get(engine);
+    d_ptr->engine->ownedAgents.append(this);
 }
 
 /*!