From 2363c63e6418d52d665ea79bec2f597145ee5021 Mon Sep 17 00:00:00 2001 From: Nick Couchman Date: Mon, 20 Mar 2017 14:55:35 -0400 Subject: [PATCH] GUACAMOLE-102: Adding some commentary for changes in the RestrictedGuacamoleTunnelService class. --- .../auth/jdbc/tunnel/RestrictedGuacamoleTunnelService.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/tunnel/RestrictedGuacamoleTunnelService.java b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/tunnel/RestrictedGuacamoleTunnelService.java index 3db47c660..b41ec91bc 100644 --- a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/tunnel/RestrictedGuacamoleTunnelService.java +++ b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/tunnel/RestrictedGuacamoleTunnelService.java @@ -190,11 +190,15 @@ public class RestrictedGuacamoleTunnelService logger.debug("Calculating weights for connections {} and {}.", a.getName(), b.getName()); int cw = 0; int weightA = a.getConnectionWeight(); + // If the weight is null, we go ahead and sort, anyway if (weightA == null) weightA = 0; + + // If the weight is null, we go ahead and sort, anyway int weightB = b.getConnectionWeight(); if (weightB == null) weightB = 0; + int connsA = getActiveConnections(a).size(); int connsB = getActiveConnections(b).size(); logger.debug("Connection {} has computed weight of {}.", a.getName(), connsA * 10000 / weightA);