You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pa...@apache.org on 2021/11/03 14:04:43 UTC

[felix-dev] branch issues/FELIX-6467 created (now 0e862d2)

This is an automated email from the ASF dual-hosted git repository.

pauls pushed a change to branch issues/FELIX-6467
in repository https://gitbox.apache.org/repos/asf/felix-dev.git.


      at 0e862d2  FELIX-6467: add a permission check to conditional permsion info commit

This branch includes the following new commits:

     new 0e862d2  FELIX-6467: add a permission check to conditional permsion info commit

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


[felix-dev] 01/01: FELIX-6467: add a permission check to conditional permsion info commit

Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

pauls pushed a commit to branch issues/FELIX-6467
in repository https://gitbox.apache.org/repos/asf/felix-dev.git

commit 0e862d2b96ba594948552e7b0a7b597f782def4d
Author: Karl Pauls <ka...@gmail.com>
AuthorDate: Wed Nov 3 15:04:31 2021 +0100

    FELIX-6467: add a permission check to conditional permsion info commit
---
 .../security/condpermadmin/ConditionalPermissionAdminImpl.java      | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/framework.security/src/main/java/org/apache/felix/framework/security/condpermadmin/ConditionalPermissionAdminImpl.java b/framework.security/src/main/java/org/apache/felix/framework/security/condpermadmin/ConditionalPermissionAdminImpl.java
index 9709424..ba6c50f 100644
--- a/framework.security/src/main/java/org/apache/felix/framework/security/condpermadmin/ConditionalPermissionAdminImpl.java
+++ b/framework.security/src/main/java/org/apache/felix/framework/security/condpermadmin/ConditionalPermissionAdminImpl.java
@@ -878,6 +878,12 @@ public final class ConditionalPermissionAdminImpl implements
 
             public boolean commit()
             {
+                Object sm = System.getSecurityManager();
+                if (sm != null)
+                {
+                    ((SecurityManager) sm).checkPermission(Permissions.ALL_PERMISSION);
+                }
+
                 synchronized (m_condPermInfos)
                 {
                     if (current.equals(new ArrayList(m_condPermInfos.values())))