From 5610268be8a366a6dc1aad09b4e23d46490b4d86 Mon Sep 17 00:00:00 2001 From: Johnny Willemsen Date: Sun, 7 Jan 2007 18:44:59 +0000 Subject: Sun Jan 7 18:44:12 UTC 2007 Johnny Willemsen --- CIAO/ChangeLog | 8 ++++++++ CIAO/DAnCE/NodeApplicationManager/URL_Parser.cpp | 3 --- CIAO/DAnCE/RepositoryManager/URL_Parser.cpp | 14 +++++--------- 3 files changed, 13 insertions(+), 12 deletions(-) diff --git a/CIAO/ChangeLog b/CIAO/ChangeLog index b62ba593da1..2589ec2c963 100644 --- a/CIAO/ChangeLog +++ b/CIAO/ChangeLog @@ -1,3 +1,11 @@ +Sun Jan 7 18:44:12 UTC 2007 Johnny Willemsen + + * DAnCE/NodeApplicationManager/URL_Parser.cpp: + No need to set members to 0 in destructor + + * DAnCE/RepositoryManager/URL_Parser.cpp: + Fixed conversion warning and memory leak + Sun Jan 7 12:14:12 UTC 2007 Johnny Willemsen * ciao/CCM_ClientContainerInterceptor.idl: diff --git a/CIAO/DAnCE/NodeApplicationManager/URL_Parser.cpp b/CIAO/DAnCE/NodeApplicationManager/URL_Parser.cpp index f79e086dd89..5d211603614 100644 --- a/CIAO/DAnCE/NodeApplicationManager/URL_Parser.cpp +++ b/CIAO/DAnCE/NodeApplicationManager/URL_Parser.cpp @@ -111,8 +111,5 @@ void URL_Parser::Error (void) URL_Parser::~URL_Parser (void) { delete [] this->hostname_; - this->hostname_ = 0; - ACE_OS::free (this->filename_); - this->filename_ = 0; } diff --git a/CIAO/DAnCE/RepositoryManager/URL_Parser.cpp b/CIAO/DAnCE/RepositoryManager/URL_Parser.cpp index 3024606da7f..37187ebff61 100644 --- a/CIAO/DAnCE/RepositoryManager/URL_Parser.cpp +++ b/CIAO/DAnCE/RepositoryManager/URL_Parser.cpp @@ -4,6 +4,7 @@ #include "ace/ARGV.h" #include "URL_Parser.h" +#include "ace/ACE.h" #include "ace/OS_NS_string.h" bool @@ -37,7 +38,7 @@ URL_Parser::parse_args (int argc, ACE_TCHAR *argv[]) } URL_Parser::URL_Parser (void) - : hostname_ ("127.0.0.1"), + : hostname_ (ACE::strnew ("127.0.0.1")), port_ (ACE_DEFAULT_HTTP_SERVER_PORT), filename_ (0), debug_ (false) @@ -69,6 +70,7 @@ bool URL_Parser::parseURL (char* url) else { size_t host_len = ptr - url; + ACE::strdelete (this->hostname_); ACE_NEW_RETURN (this->hostname_, char [host_len + 1], false); ACE_OS::strncpy (this->hostname_, url, host_len); this->hostname_ [host_len] = '\0'; @@ -94,12 +96,6 @@ void URL_Parser::Error (void) URL_Parser::~URL_Parser() { - if(this->hostname_) - { - delete [] this->hostname_; - } - if (this->filename_) - { - ACE_OS::free (this->filename_); - } + delete [] this->hostname_; + ACE_OS::free (this->filename_); } -- cgit v1.2.1