maxConnectionsPerUserForTask: Fix NPE Fix NPE in cases where maxConnectionsPerUserForTask specified in a non global section of quota config. Change-Id: Ic680cd2e338667c14dcac08ecdd92e3e439afdc6
diff --git a/src/main/java/com/googlesource/gerrit/plugins/quota/MaxConnectionsLimiter.java b/src/main/java/com/googlesource/gerrit/plugins/quota/MaxConnectionsLimiter.java index f10f07a..a5245ec 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/quota/MaxConnectionsLimiter.java +++ b/src/main/java/com/googlesource/gerrit/plugins/quota/MaxConnectionsLimiter.java
@@ -69,9 +69,11 @@ Config cfg = projectCache.getAllProjects().getConfig("quota.config").get(); for (String group : cfg.getSubsections(AccountLimitsConfig.GROUP_SECTION)) { String val = cfg.getString(AccountLimitsConfig.GROUP_SECTION, group, CONFIG_KEY); - Matcher matcher = REST_API_REGEX.matcher(val); - if (matcher.matches()) { - limits.add(new Limit(group, Integer.parseInt(matcher.group(1)))); + if (val != null) { + Matcher matcher = REST_API_REGEX.matcher(val); + if (matcher.matches()) { + limits.add(new Limit(group, Integer.parseInt(matcher.group(1)))); + } } }