aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #1262 from ctrlaltca/servatrice_mail

Browse source 
Merge memory leak fix for smtp client
This commit is contained in:
Zach 2015-07-15 12:24:23 -04:00
commit 97acbbfd41
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
servatrice/src/smtp/smtpclient.cpp
View file

@ -25,6 +25,7 @@
/* [1] Constructors and destructors */ /* [1] Constructors and destructors */
SmtpClient::SmtpClient(const QString & host, int port, ConnectionType connectionType) : SmtpClient::SmtpClient(const QString & host, int port, ConnectionType connectionType) :
socket(NULL),
name("localhost"), name("localhost"),
authMethod(AuthPlain), authMethod(AuthPlain),
connectionTimeout(5000), connectionTimeout(5000),
@ -44,7 +45,10 @@ SmtpClient::SmtpClient(const QString & host, int port, ConnectionType connection
this, SLOT(socketReadyRead())); this, SLOT(socketReadyRead()));
} }
SmtpClient::~SmtpClient() {} SmtpClient::~SmtpClient() {
if (socket)
delete socket;
}
/* [1] --- */ /* [1] --- */
@ -80,6 +84,9 @@ void SmtpClient::setConnectionType(ConnectionType ct)
{ {
this->connectionType = ct; this->connectionType = ct;
if (socket)
delete socket;
switch (connectionType) switch (connectionType)
{ {
case TcpConnection: case TcpConnection: