This is an automated email from the ASF dual-hosted git repository.
rmaucher pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 3eca9b1a17 Minor fixes from code review
3eca9b1a17 is described below
commit 3eca9b1a177e96ce81e482b586a959e447ea704a
Author: remm <[email protected]>
AuthorDate: Wed Jun 17 16:02:38 2026 +0200
Minor fixes from code review
---
java/jakarta/websocket/server/ServerEndpointConfig.java | 2 +-
java/org/apache/catalina/core/StandardServer.java | 2 +-
java/org/apache/catalina/tribes/group/AbsoluteOrder.java | 2 +-
java/org/apache/el/parser/AstIdentifier.java | 2 +-
java/org/apache/tomcat/websocket/WsFrameBase.java | 3 ++-
5 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/java/jakarta/websocket/server/ServerEndpointConfig.java
b/java/jakarta/websocket/server/ServerEndpointConfig.java
index 78dbba789e..8df2798fc3 100644
--- a/java/jakarta/websocket/server/ServerEndpointConfig.java
+++ b/java/jakarta/websocket/server/ServerEndpointConfig.java
@@ -115,7 +115,7 @@ public interface ServerEndpointConfig extends
EndpointConfig {
this.path = path;
}
- /**
+ /**
* Builds and returns a {@link ServerEndpointConfig} with the
configured settings.
*
* @return The configured server endpoint configuration
diff --git a/java/org/apache/catalina/core/StandardServer.java
b/java/org/apache/catalina/core/StandardServer.java
index 068264549d..d54acda7ce 100644
--- a/java/org/apache/catalina/core/StandardServer.java
+++ b/java/org/apache/catalina/core/StandardServer.java
@@ -831,7 +831,7 @@ public final class StandardServer extends
LifecycleMBeanBase implements Server {
ObjectName sname = new ObjectName("Catalina:type=StoreConfig");
MBeanServer server = Registry.getRegistry(null).getMBeanServer();
if (server.isRegistered(sname)) {
- server.invoke(sname, "store", new Object[] { context }, new
String[] { "java.lang.String" });
+ server.invoke(sname, "store", new Object[] { context }, new
String[] { "org.apache.catalina.Context" });
} else {
log.error(sm.getString("standardServer.storeConfig.notAvailable", sname));
}
diff --git a/java/org/apache/catalina/tribes/group/AbsoluteOrder.java
b/java/org/apache/catalina/tribes/group/AbsoluteOrder.java
index 494b8cc93c..441adc3623 100644
--- a/java/org/apache/catalina/tribes/group/AbsoluteOrder.java
+++ b/java/org/apache/catalina/tribes/group/AbsoluteOrder.java
@@ -169,7 +169,7 @@ public class AbsoluteOrder {
* @return comparison result
*/
protected int compareBytes(byte b1, byte b2) {
- return compareInts(b1, b2);
+ return compareInts(b1 & 0xFF, b2 & 0xFF);
}
/**
diff --git a/java/org/apache/el/parser/AstIdentifier.java
b/java/org/apache/el/parser/AstIdentifier.java
index a02b10d105..d1f1eaea98 100644
--- a/java/org/apache/el/parser/AstIdentifier.java
+++ b/java/org/apache/el/parser/AstIdentifier.java
@@ -92,7 +92,7 @@ public final class AstIdentifier extends SimpleNode {
} finally {
// Always reset the flag to false so the optimisation is not
applied
// inappropriately
- ctx.putContext(this.getClass(), Boolean.FALSE);
+ ctx.putContext(ELResolver.StandaloneIdentifierMarker.class,
Boolean.FALSE);
}
if (ctx.isPropertyResolved()) {
diff --git a/java/org/apache/tomcat/websocket/WsFrameBase.java
b/java/org/apache/tomcat/websocket/WsFrameBase.java
index 2d8dcf6bba..17463fa89b 100644
--- a/java/org/apache/tomcat/websocket/WsFrameBase.java
+++ b/java/org/apache/tomcat/websocket/WsFrameBase.java
@@ -357,7 +357,8 @@ public abstract class WsFrameBase {
new CloseReason(CloseCodes.PROTOCOL_ERROR,
sm.getString("wsFrame.oneByteCloseCode")));
}
if (controlBufferBinary.remaining() > 1) {
- code = controlBufferBinary.getShort();
+ short rawCode = controlBufferBinary.getShort();
+ code = rawCode & 0xFFFF;
if (controlBufferBinary.remaining() > 0) {
CoderResult cr =
utf8DecoderControl.decode(controlBufferBinary, controlBufferText, true);
if (cr.isError()) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
