Browse Source

Merge pull request #5881 from Chocobo1/trackerfilter_crash

Fix crash in TrackerFiltersList::removeItem()
adaptive-webui-19844
sledgehammer999 8 years ago committed by GitHub
parent
commit
05387edbfa
  1. 7
      src/gui/transferlistfilterswidget.cpp

7
src/gui/transferlistfilterswidget.cpp

@ -523,7 +523,7 @@ void TrackerFiltersList::addItem(const QString &tracker, const QString &hash) @@ -523,7 +523,7 @@ void TrackerFiltersList::addItem(const QString &tracker, const QString &hash)
void TrackerFiltersList::removeItem(const QString &tracker, const QString &hash)
{
QString host = getHost(tracker);
QListWidgetItem *trackerItem = 0;
QListWidgetItem *trackerItem = nullptr;
QStringList tmp = m_trackers.value(host);
int row = 0;
@ -531,7 +531,7 @@ void TrackerFiltersList::removeItem(const QString &tracker, const QString &hash) @@ -531,7 +531,7 @@ void TrackerFiltersList::removeItem(const QString &tracker, const QString &hash)
return;
tmp.removeAll(hash);
if (host != "") {
if (!host.isEmpty()) {
// Remove from 'Error' and 'Warning' view
trackerSuccess(hash, tracker);
row = rowFromTracker(host);
@ -544,7 +544,8 @@ void TrackerFiltersList::removeItem(const QString &tracker, const QString &hash) @@ -544,7 +544,8 @@ void TrackerFiltersList::removeItem(const QString &tracker, const QString &hash)
updateGeometry();
return;
}
trackerItem->setText(tr("%1 (%2)", "openbittorrent.com (10)").arg(host).arg(tmp.size()));
if (trackerItem != nullptr)
trackerItem->setText(tr("%1 (%2)", "openbittorrent.com (10)").arg(host).arg(tmp.size()));
}
else {
row = 1;

Loading…
Cancel
Save