fix: properly aggregate tracker-level numFails by skipping -1 sentinel

LWWZH · LWWZH · commit 7440918350c6 · 2026-05-27T07:31:00.000+10:00
diff --git a/src/base/bittorrent/torrentimpl.cpp b/src/base/bittorrent/torrentimpl.cpp
@@ -261,7 +261,8 @@ namespace
             trackerEntryStatus.numSeeds = std::max(trackerEntryStatus.numSeeds, endpointStatus.numSeeds);
             trackerEntryStatus.numLeeches = std::max(trackerEntryStatus.numLeeches, endpointStatus.numLeeches);
             trackerEntryStatus.numDownloaded = std::max(trackerEntryStatus.numDownloaded, endpointStatus.numDownloaded);
-            trackerEntryStatus.numFails = std::min(trackerEntryStatus.numFails, endpointStatus.numFails);
+            if (endpointStatus.numFails >= 0)
+                trackerEntryStatus.numFails = (trackerEntryStatus.numFails < 0) ? endpointStatus.numFails : std::min(trackerEntryStatus.numFails, endpointStatus.numFails);
 
             if (endpointStatus.state == trackerEntryStatus.state)
             {

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,8 @@ namespace`
`261`	`261`	`trackerEntryStatus.numSeeds = std::max(trackerEntryStatus.numSeeds, endpointStatus.numSeeds);`
`262`	`262`	`trackerEntryStatus.numLeeches = std::max(trackerEntryStatus.numLeeches, endpointStatus.numLeeches);`
`263`	`263`	`trackerEntryStatus.numDownloaded = std::max(trackerEntryStatus.numDownloaded, endpointStatus.numDownloaded);`
`264`		`- trackerEntryStatus.numFails = std::min(trackerEntryStatus.numFails, endpointStatus.numFails);`
	`264`	`+ if (endpointStatus.numFails >= 0)`
	`265`	`+ trackerEntryStatus.numFails = (trackerEntryStatus.numFails < 0) ? endpointStatus.numFails : std::min(trackerEntryStatus.numFails, endpointStatus.numFails);`
`265`	`266`
`266`	`267`	`if (endpointStatus.state == trackerEntryStatus.state)`
`267`	`268`	`{`