mirror/bitcoin
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant nullptr checks before deallocation

Browse Source 
Rationale:
* delete ptr is a no-op if ptr is nullptr
This commit is contained in:
practicalswift committed 2017-08-14 17:45:15 +02:00
1 parent 140de14a12
commit b109a1c396
3 files changed
-3
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

src/net.cpp
-1
View File
@@ -2753,7 +2753,6 @@ CNode::~CNode()
{ {
CloseSocket(hSocket); CloseSocket(hSocket);
if (pfilter)
delete pfilter; delete pfilter;
} }
src/qt/paymentrequestplus.cpp
-1
View File
@@ -194,7 +194,6 @@ bool PaymentRequestPlus::getMerchant(X509_STORE* certStore, QString& merchant) c
qWarning() << "PaymentRequestPlus::getMerchant: SSL error: " << err.what(); qWarning() << "PaymentRequestPlus::getMerchant: SSL error: " << err.what();
} }
if (website)
delete[] website; delete[] website;
X509_STORE_CTX_free(store_ctx); X509_STORE_CTX_free(store_ctx);
for (unsigned int i = 0; i < certs.size(); i++) for (unsigned int i = 0; i < certs.size(); i++)
src/qt/paymentserver.cpp
-1
View File
@@ -364,7 +364,6 @@ void PaymentServer::initNetManager()
{ {
if (!optionsModel) if (!optionsModel)
return; return;
if (netManager != nullptr)
delete netManager; delete netManager;
// netManager is used to fetch paymentrequests given in bitcoin: URIs // netManager is used to fetch paymentrequests given in bitcoin: URIs