From dbb62ded7784459a903197db6ff8becd5b426e69 Mon Sep 17 00:00:00 2001 From: Nick Couchman Date: Mon, 6 Feb 2017 12:01:14 -0500 Subject: [PATCH] GUACAMOLE-197: You can't throw something after you return out of the function, so lets remove the return statement. --- .../guacamole/auth/radius/AuthenticationProviderService.java | 1 - 1 file changed, 1 deletion(-) diff --git a/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java b/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java index f77f432e3..c6650bc8b 100644 --- a/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java +++ b/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java @@ -131,7 +131,6 @@ public class AuthenticationProviderService { catch(UnknownAttributeException e) { logger.error("Error in talks with RADIUS server."); logger.debug("RADIUS challenged by didn't provide right attributes."); - return null; throw new GuacamoleInvalidCredentialsException("Authentication error.", CredentialsInfo.USERNAME_PASSWORD); } }