libqtappfw: message: fix memory leak with QJsonArray

QJsonArray object was being allocated on the heap and never freed.
Switch to having a m_request member field and confirming it is only
populated with data once.

Change-Id: I60f406eff032650cac878efa7e1db61ee18e2eb2
Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>

1 files changed, 10 insertions, 6 deletions

diff --git a/message.cpp b/message.cpp
index 95bc586..33211da 100644
--- a/message.cpp
+++ b/message.cpp
@@ -29,14 +29,18 @@ Message::Message()
 
 bool Message::createRequest(QString api, QString verb, QJsonValue parameter)
 {
-	QJsonArray *request = new QJsonArray;
-	request->append(Call);
-	request->append(9999);
-	request->append(api + (QString)("/") + verb);
-	request->append(QJsonValue(parameter));
+	if (!m_request.isEmpty()){
+		qWarning("Message instance has already been used. Cannot send another request.");
+		return false;
+	}
+
+	m_request.append(Call);
+	m_request.append(9999);
+	m_request.append(api + (QString)("/") + verb);
+	m_request.append(QJsonValue(parameter));
 
 	QJsonDocument jdoc;
-	jdoc.setArray(*request);
+	jdoc.setArray(m_request);
 
 	m_jdoc = jdoc;
 	m_init = true;