[Libreoffice-commits] .: automation/source
Thomas Arnhold
tarnhold at kemper.freedesktop.org
Sat Jul 30 06:30:05 PDT 2011
automation/source/testtool/httprequest.cxx | 9 +++++++++
automation/source/testtool/httprequest.hxx | 1 +
automation/source/testtool/objtest.cxx | 1 +
3 files changed, 11 insertions(+)
New commits:
commit f73dd26567c5e0e597f4464901148d7060b9875f
Author: Thomas Arnhold <thomas at arnhold.org>
Date: Sat Jul 30 13:33:38 2011 +0200
callcatcher: bring HttpRequest dtor back
This partially reverts 7cb18732923040b27ebd0d1f9c9bc01766d04d59
and fixes a potential leak.
diff --git a/automation/source/testtool/httprequest.cxx b/automation/source/testtool/httprequest.cxx
index b99cc72..f33c9a0 100644
--- a/automation/source/testtool/httprequest.cxx
+++ b/automation/source/testtool/httprequest.cxx
@@ -50,6 +50,15 @@ HttpRequest::HttpRequest()
{
}
+HttpRequest::~HttpRequest()
+{
+ delete pStream;
+ pStream = NULL;
+
+ delete pOutSocket;
+ pOutSocket = NULL;
+}
+
void HttpRequest::SetRequest( rtl::OString aHost, rtl::OString aPath, sal_uInt16 nPort )
{
nStatus = HTTP_REQUEST_SET;
diff --git a/automation/source/testtool/httprequest.hxx b/automation/source/testtool/httprequest.hxx
index 90e97c4..438dba8 100644
--- a/automation/source/testtool/httprequest.hxx
+++ b/automation/source/testtool/httprequest.hxx
@@ -65,6 +65,7 @@ class HttpRequest
void Init();
public:
HttpRequest();
+ ~HttpRequest();
void SetRequest( rtl::OString aHost, rtl::OString aPath, sal_uInt16 nPort );
void SetProxy( rtl::OString aHost, sal_uInt16 nPort );
diff --git a/automation/source/testtool/objtest.cxx b/automation/source/testtool/objtest.cxx
index 050458e..d1ae1bf 100644
--- a/automation/source/testtool/objtest.cxx
+++ b/automation/source/testtool/objtest.cxx
@@ -740,6 +740,7 @@ TestToolObj::~TestToolObj()
if ( pImpl->pTTSfxBroadcaster )
delete pImpl->pTTSfxBroadcaster;
delete pImpl->pChildEnv;
+ delete pImpl->pHttpRequest;
pImpl->xErrorList.Clear();
pImpl->xWarningList.Clear();
More information about the Libreoffice-commits
mailing list