Return-Path: Delivered-To: apmail-db-jdo-dev-archive@www.apache.org Received: (qmail 36655 invoked from network); 15 Jul 2008 01:02:25 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 15 Jul 2008 01:02:25 -0000 Received: (qmail 90785 invoked by uid 500); 15 Jul 2008 01:02:24 -0000 Mailing-List: contact jdo-dev-help@db.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jdo-dev@db.apache.org Delivered-To: mailing list jdo-dev@db.apache.org Received: (qmail 90774 invoked by uid 99); 15 Jul 2008 01:02:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 14 Jul 2008 18:02:24 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 15 Jul 2008 01:01:40 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 20DA6234C16C for ; Mon, 14 Jul 2008 18:01:34 -0700 (PDT) Message-ID: <1169441726.1216083694133.JavaMail.jira@brutus> Date: Mon, 14 Jul 2008 18:01:34 -0700 (PDT) From: "Craig Russell (JIRA)" To: jdo-dev@db.apache.org Subject: [jira] Updated: (JDO-555) All calls to Class.getMethod and Method.invoke (among others) need to be invoked inside a doPrivileged block. In-Reply-To: <20713061.1196103042989.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/JDO-555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Craig Russell updated JDO-555: ------------------------------ Attachment: xmlbean.patch This patch adds doPrivileged to calls to spring bean processing. The method getBean is the method that is invoked in doPrivileged. The same method is used in QueryTest and in CompletenessTest. It might appear in other places in the tck as well. So, should we put getBean into JDO_Test? Create a new subclass of JDO_Test that would be inherited by QueryTest and CompletenessTest? I don't like the duplication of the method in two places. Another issue is the treatment of the root list. In QueryTest, two different class implementations both use the same pattern: call the root list bean "root". This patch assumes that getRootList which has no actual common implementation between the MylibReader and CompanyReader, simply returns the xml bean called "root". We could also create a common BeanReader class that both MylibReader and CompanyReader inherit. Suggestions? > All calls to Class.getMethod and Method.invoke (among others) need to be invoked inside a doPrivileged block. > ------------------------------------------------------------------------------------------------------------- > > Key: JDO-555 > URL: https://issues.apache.org/jira/browse/JDO-555 > Project: JDO > Issue Type: Bug > Components: api2, api2-legacy > Affects Versions: JDO 2 maintenance release 1 > Reporter: Matthew T. Adams > Assignee: Andy Jefferson > Fix For: JDO 2 maintenance release 2 > > Attachments: datanucleus.patch, datanucleus.patch, jdo-555.patch, jdo-555.patch, jdo-555.patch, xmlbean.patch > > > Discovered in review of patch to JDO-545. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.