[tomcat] branch main updated: Use consistent method for manager access

[remm]

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

remm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/main by this push:
     new fd2b06de2c Use consistent method for manager access
fd2b06de2c is described below

commit fd2b06de2c35ce7aecd67ef2928449cbc62e625f
Author: remm <r...@apache.org>
AuthorDate: Fri Sep 8 10:56:14 2023 +0200

    Use consistent method for manager access
    
    Found by coverity (for wrong reasons, but there's no reason not to
    change it).
---
 java/org/apache/catalina/ha/session/JvmRouteBinderValve.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java 
b/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java
index c55bfa2eee..a5e5955d30 100644
--- a/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java
+++ b/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java
@@ -301,7 +301,7 @@ public class JvmRouteBinderValve extends ValveBase 
implements ClusterValve {
      */
     protected void changeSessionID(Request request, String sessionId, String 
newSessionID, Session catalinaSession) {
         fireLifecycleEvent("Before session migration", catalinaSession);
-        catalinaSession.getManager().changeSessionId(catalinaSession, 
newSessionID);
+        getManager(request).changeSessionId(catalinaSession, newSessionID);
         changeRequestSessionID(request, sessionId, newSessionID);
         changeSessionAuthenticationNote(sessionId, newSessionID, 
catalinaSession);
         fireLifecycleEvent("After session migration", catalinaSession);


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