Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
From: bugzilla@apache.org
To: dev@tomcat.apache.org
Subject: DO NOT REPLY [Bug 51472] New: Bugin ? in BeanELResolver.getMethod()
Date: Tue, 05 Jul 2011 02:49:05 +0000
Message-ID: <bug-51472-78@https.issues.apache.org/bugzilla/>
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0

https://issues.apache.org/bugzilla/show_bug.cgi?id=51472

             Bug #: 51472
           Summary: Bugin ? in BeanELResolver.getMethod()
           Product: Tomcat 7
           Version: 7.0.16
          Platform: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Servlet & JSP API
        AssignedTo: dev@tomcat.apache.org
        ReportedBy: friedhelm.kuehn@cms-it-services.de
    Classification: Unclassified


BeanResolver.getMethod() starts with:
        if (m == null || Modifier.isPublic(type.getModifiers())) {
            return m;
        }

That tests method existence and whether Class has public modifier.
I think that public Modifier of Method should be tested instead/or
additionally.

So:
        if (m == null || Modifier.isPublic(m.getModifiers())) {
            return m;
        }

Or: 
        if (m == null || Modifier.isPublic(type.getModifiers()
                      && Modifier.isPublic(m.getModifiers())) {
            return m;
        }

Right ?

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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