Browse Source

Remove redundant nullptr checks before deallocation

Rationale:
* delete ptr is a no-op if ptr is nullptr
0.16
practicalswift 7 years ago
parent
commit
b109a1c396
  1. 3
      src/net.cpp
  2. 3
      src/qt/paymentrequestplus.cpp
  3. 3
      src/qt/paymentserver.cpp

3
src/net.cpp

@ -2753,8 +2753,7 @@ CNode::~CNode() @@ -2753,8 +2753,7 @@ CNode::~CNode()
{
CloseSocket(hSocket);
if (pfilter)
delete pfilter;
delete pfilter;
}
void CNode::AskFor(const CInv& inv)

3
src/qt/paymentrequestplus.cpp

@ -194,8 +194,7 @@ bool PaymentRequestPlus::getMerchant(X509_STORE* certStore, QString& merchant) c @@ -194,8 +194,7 @@ bool PaymentRequestPlus::getMerchant(X509_STORE* certStore, QString& merchant) c
qWarning() << "PaymentRequestPlus::getMerchant: SSL error: " << err.what();
}
if (website)
delete[] website;
delete[] website;
X509_STORE_CTX_free(store_ctx);
for (unsigned int i = 0; i < certs.size(); i++)
X509_free(certs[i]);

3
src/qt/paymentserver.cpp

@ -364,8 +364,7 @@ void PaymentServer::initNetManager() @@ -364,8 +364,7 @@ void PaymentServer::initNetManager()
{
if (!optionsModel)
return;
if (netManager != nullptr)
delete netManager;
delete netManager;
// netManager is used to fetch paymentrequests given in bitcoin: URIs
netManager = new QNetworkAccessManager(this);

Loading…
Cancel
Save