[tomcat] 01/02: StandardSession.java @doWriteObject: adds null-check to sessionPrincipal before checking if serializable to prevent misleading warning message logged.

[markt]

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

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git

commit d2e0ef68de5408572fdb9885b2a0be5458525613
Author: Yordanov <tyordanov@amc.intra>
AuthorDate: Thu Jul 13 12:04:25 2023 +0200

    StandardSession.java @doWriteObject: adds null-check to sessionPrincipal 
before checking if serializable to prevent misleading warning message logged.
---
 java/org/apache/catalina/session/StandardSession.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/java/org/apache/catalina/session/StandardSession.java 
b/java/org/apache/catalina/session/StandardSession.java
index f6c529a20c..5e1f463da8 100644
--- a/java/org/apache/catalina/session/StandardSession.java
+++ b/java/org/apache/catalina/session/StandardSession.java
@@ -1602,7 +1602,7 @@ public class StandardSession implements HttpSession, 
Session, Serializable {
         if (getPersistAuthentication()) {
             sessionAuthType = getAuthType();
             sessionPrincipal = getPrincipal();
-            if (!(sessionPrincipal instanceof Serializable)) {
+            if (sessionPrincipal != null && !(sessionPrincipal instanceof 
Serializable)) {
                 sessionPrincipal = null;
                 
manager.getContext().getLogger().warn(sm.getString("standardSession.principalNotSerializable",
 id));
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org