Fix UpstreamPolicie: use size_t when counting for cur_idx. (#1809)

holmes1412 · web-flow · commit 8dc650227603 · 2026-04-01T21:33:59.000+08:00
diff --git a/src/nameservice/UpstreamPolicies.cc b/src/nameservice/UpstreamPolicies.cc
@@ -612,8 +612,8 @@ void UPSWeightedRandomPolicy::fuse_one_server(const EndpointAddress *addr)
 EndpointAddress *UPSVNSWRRPolicy::first_strategy(const ParsedURI& uri,
 												 WFNSTracing *tracing)
 {
-	int idx = this->cur_idx.fetch_add(1);
-	int pos = 0;
+	size_t idx = this->cur_idx.fetch_add(1);
+	size_t pos = 0;
 	for (int i = 0; i < this->total_weight; i++, idx++)
 	{
 		pos = this->pre_generated_vec[idx % this->pre_generated_vec.size()];

Original file line number	Diff line number	Diff line change
`@@ -612,8 +612,8 @@ void UPSWeightedRandomPolicy::fuse_one_server(const EndpointAddress *addr)`
`612`	`612`	`EndpointAddress *UPSVNSWRRPolicy::first_strategy(const ParsedURI& uri,`
`613`	`613`	`WFNSTracing *tracing)`
`614`	`614`	`{`
`615`		`- int idx = this->cur_idx.fetch_add(1);`
`616`		`- int pos = 0;`
	`615`	`+ size_t idx = this->cur_idx.fetch_add(1);`
	`616`	`+ size_t pos = 0;`
`617`	`617`	`for (int i = 0; i < this->total_weight; i++, idx++)`
`618`	`618`	`{`
`619`	`619`	`pos = this->pre_generated_vec[idx % this->pre_generated_vec.size()];`