This is an automated email from the ASF dual-hosted git repository.
markt-asf pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new ff430c882e Follow-up to o.a.c.realm code review
ff430c882e is described below
commit ff430c882e68ab86087f10dc9ce9cf3ef110e296
Author: Mark Thomas <[email protected]>
AuthorDate: Mon Jun 15 20:53:38 2026 +0100
Follow-up to o.a.c.realm code review
---
java/org/apache/catalina/realm/JAASRealm.java | 3 +--
java/org/apache/catalina/realm/JNDIRealm.java | 7 +++++--
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/java/org/apache/catalina/realm/JAASRealm.java
b/java/org/apache/catalina/realm/JAASRealm.java
index 00b71ebff8..fb913082f4 100644
--- a/java/org/apache/catalina/realm/JAASRealm.java
+++ b/java/org/apache/catalina/realm/JAASRealm.java
@@ -632,16 +632,15 @@ public class JAASRealm extends RealmBase {
try (ConfigurationSource.Resource configFileResource =
ConfigFileLoader.getSource().getResource(configFile)) {
config =
constructor.newInstance(configFileResource.getURI());
- this.jaasConfigurationLoaded = true;
} catch (IOException ioe) {
throw new
RuntimeException(sm.getString("jaasRealm.configFileNotFound", configFile));
}
} else {
URI uri = resource.toURI();
config = constructor.newInstance(uri);
- this.jaasConfigurationLoaded = true;
}
this.jaasConfiguration = config;
+ this.jaasConfigurationLoaded = true;
return this.jaasConfiguration;
}
} catch (InvocationTargetException ex) {
diff --git a/java/org/apache/catalina/realm/JNDIRealm.java
b/java/org/apache/catalina/realm/JNDIRealm.java
index a1e83c2746..25c9420361 100644
--- a/java/org/apache/catalina/realm/JNDIRealm.java
+++ b/java/org/apache/catalina/realm/JNDIRealm.java
@@ -19,6 +19,7 @@ package org.apache.catalina.realm;
import java.io.IOException;
import java.net.URI;
import java.net.URISyntaxException;
+import java.nio.charset.StandardCharsets;
import java.security.KeyManagementException;
import java.security.NoSuchAlgorithmException;
import java.security.Principal;
@@ -2323,8 +2324,10 @@ public class JNDIRealm extends RealmBase {
while (e.hasMore()) {
Object value = e.next();
String valueString;
- if (value instanceof byte[]) {
- valueString = new String((byte[]) value);
+ if (value == null) {
+ valueString = null;
+ } else if (value instanceof byte[]) {
+ valueString = new String((byte[]) value,
StandardCharsets.UTF_8);
} else {
valueString = value.toString();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
