merge bitcoin-core/gui#717: Use the stored CSubNet entry when unbanning

kwvg · kwvg · commit 9385d44bc62e · 2025-05-27T09:27:01.000Z
diff --git a/src/qt/bantablemodel.cpp b/src/qt/bantablemodel.cpp
@@ -43,7 +43,7 @@ class BanTablePriv
     /** Order (ascending or descending) to sort nodes by */
     Qt::SortOrder sortOrder;
 
-    /** Pull a full list of banned nodes from CNode into our cache */
+    /** Pull a full list of banned nodes from interfaces::Node into our cache */
     void refreshBanlist(interfaces::Node& node)
     {
         banmap_t banMap;
@@ -181,3 +181,9 @@ bool BanTableModel::shouldShow()
 {
     return priv->size() > 0;
 }
+
+bool BanTableModel::unban(const QModelIndex& index)
+{
+    CCombinedBan* ban{static_cast<CCombinedBan*>(index.internalPointer())};
+    return ban != nullptr && m_node.unban(ban->subnet);
+}
diff --git a/src/qt/bantablemodel.h b/src/qt/bantablemodel.h
@@ -37,7 +37,7 @@ class BannedNodeLessThan
 };
 
 /**
-   Qt model providing information about connected peers, similar to the
+   Qt model providing information about banned peers, similar to the
    "getpeerinfo" RPC call. Used by the rpc console UI.
  */
 class BanTableModel : public QAbstractTableModel
@@ -68,6 +68,8 @@ class BanTableModel : public QAbstractTableModel
 
     bool shouldShow();
 
+    bool unban(const QModelIndex& index);
+
 public Q_SLOTS:
     void refresh();
 
diff --git a/src/qt/rpcconsole.cpp b/src/qt/rpcconsole.cpp
@@ -14,7 +14,6 @@
 
 #include <chainparams.h>
 #include <interfaces/node.h>
-#include <netbase.h>
 #include <node/connection_types.h>
 #include <qt/bantablemodel.h>
 #include <qt/clientmodel.h>
@@ -1492,15 +1491,13 @@ void RPCConsole::unbanSelectedNode()
 
     // Get selected ban addresses
     QList<QModelIndex> nodes = GUIUtil::getEntryData(ui->banlistWidget, BanTableModel::Address);
-    for(int i = 0; i < nodes.count(); i++)
-    {
-        // Get currently selected ban address
-        QString strNode = nodes.at(i).data().toString();
-        CSubNet possibleSubnet{LookupSubNet(strNode.toStdString())};
-        if (possibleSubnet.IsValid() && m_node.unban(possibleSubnet))
-        {
-            clientModel->getBanTableModel()->refresh();
-        }
+    BanTableModel* ban_table_model{clientModel->getBanTableModel()};
+    bool unbanned{false};
+    for (const auto& node_index : nodes) {
+        unbanned |= ban_table_model->unban(node_index);
+    }
+    if (unbanned) {
+        ban_table_model->refresh();
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ class BanTablePriv`
`43`	`43`	`/** Order (ascending or descending) to sort nodes by */`
`44`	`44`	`Qt::SortOrder sortOrder;`
`45`	`45`
`46`		`- /** Pull a full list of banned nodes from CNode into our cache */`
	`46`	`+ /** Pull a full list of banned nodes from interfaces::Node into our cache */`
`47`	`47`	`void refreshBanlist(interfaces::Node& node)`
`48`	`48`	`{`
`49`	`49`	`banmap_t banMap;`
`@@ -181,3 +181,9 @@ bool BanTableModel::shouldShow()`
`181`	`181`	`{`
`182`	`182`	`return priv->size() > 0;`
`183`	`183`	`}`
	`184`	`+`
	`185`	`+bool BanTableModel::unban(const QModelIndex& index)`
	`186`	`+{`
	`187`	`+ CCombinedBan* ban{static_cast<CCombinedBan*>(index.internalPointer())};`
	`188`	`+ return ban != nullptr && m_node.unban(ban->subnet);`
	`189`	`+}`