This is an automated email from the ASF dual-hosted git repository.

liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/master by this push:
     new d4fd88122 [#4716]Fix load balance rule may not properly handle integer 
overflow (#4718)
d4fd88122 is described below

commit d4fd88122a606a9d62791fbd37e2f2c8b82816e2
Author: SweetWuXiaoMei <[email protected]>
AuthorDate: Mon Feb 17 08:48:46 2025 +0800

    [#4716]Fix load balance rule may not properly handle integer overflow 
(#4718)
---
 .../src/main/java/org/apache/servicecomb/loadbalance/RandomRuleExt.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/RandomRuleExt.java
 
b/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/RandomRuleExt.java
index 5f8d26c0f..318b45f38 100644
--- 
a/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/RandomRuleExt.java
+++ 
b/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/RandomRuleExt.java
@@ -31,7 +31,7 @@ public class RandomRuleExt implements RuleExt {
     if (servers.isEmpty()) {
       return null;
     }
-    int index = Math.abs(ThreadLocalRandom.current().nextInt()) % 
servers.size();
+    int index = ThreadLocalRandom.current().nextInt(servers.size());
     return servers.get(index);
   }
 }

Reply via email to