From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Tue, 5 Jul 2011 10:03:06 +0000 (+0000)
Subject: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51472
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=a60a49bb8d22863af416ce33fef8372a019cddc2;p=tomcat7.0

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51472
Correctly test modifiers when resolving bean methods with the BeanELResolver.
Patch provided by Friedhelm Kuehn.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1142953 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/javax/el/BeanELResolver.java b/java/javax/el/BeanELResolver.java
index 7ba9ce02e..c525bb5e3 100644
--- a/java/javax/el/BeanELResolver.java
+++ b/java/javax/el/BeanELResolver.java
@@ -312,7 +312,7 @@ public class BeanELResolver extends ELResolver {
     }
 
     private static final Method getMethod(Class<?> type, Method m) {
-        if (m == null || Modifier.isPublic(type.getModifiers())) {
+        if (m == null || Modifier.isPublic(m.getModifiers())) {
             return m;
         }
         Class<?>[] inf = type.getInterfaces();
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index ce1052053..1948706fd 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -84,6 +84,14 @@
       </fix>
     </changelog>
   </subsection>
+  <subsection name="Jasper">
+    <changelog>
+      <fix>
+        <bug>51472</bug>: Correctly test modifiers when resolving bean methods
+        with the BeanELResolver. Patch provided by Friedhelm Kuehn. (markt)
+      </fix>
+    </changelog>
+  </subsection>
   <subsection name="Other">
     <changelog>
       <fix>