This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit ebc1ed63be1af95ec88dfad0cd0e54d7a5a3013e
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Mon Nov 27 13:47:03 2023 +0000
Add comment on why >= rather than == is used
---
java/jakarta/el/CompositeELResolver.java | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/java/jakarta/el/CompositeELResolver.java
b/java/jakarta/el/CompositeELResolver.java
index 77331f5b6e..5c931dfe05 100644
--- a/java/jakarta/el/CompositeELResolver.java
+++ b/java/jakarta/el/CompositeELResolver.java
@@ -43,6 +43,10 @@ public class CompositeELResolver extends ELResolver {
public void add(ELResolver elResolver) {
Objects.requireNonNull(elResolver);
+ /*
+ * resolversSize should never be larger than resolvers.length. If it
ever is, the code will fail when execution
+ * reaches System.arraycopy with an IndexOutOfBoundsException.
+ */
if (resolversSize >= resolvers.length) {
ELResolver[] nr = new ELResolver[resolversSize * 2];
System.arraycopy(resolvers, 0, nr, 0, resolversSize);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
