Details

    • Type: Feature Request
    • Status: Open (View Workflow)
    • Priority: Critical
    • Resolution: Unresolved
    • Affects Version/s: 1.2.Final
    • Fix Version/s: 2.1 (Discussion)
    • Component/s: None
    • Labels:
      None
    • Sprint:
      Sprint 1

      Description

      Currently, the specification does not require any security checks to be performed when CDI SPI is used. It is not clear how this should be implemented.

      Note that since a CDI implementation is required to support bypassing platform securiy checks (e.g. when invoking aprivate observer method), total absence of security checks on CDI SPI level could result in privilege escalation. Here's an example:

      Suppose the application server bundles a sensitive method:

      public class ApplicationServer {
      
          static boolean explode() {
              // ...
          }
      }
      

      The method is package-private and thus cannot be by an application. If the application would be to invoke this method using reflection, it would need to first obtain the Method handle for the explode method. Obtaining a Method handle requires the "accessDeclaredMembers" runtime permission which the application does not have. It can however use the CDI SPI:

      Method method = null;
      for (AnnotatedMethod<?> annotatedMethod : manager.createAnnotatedType(ApplicationServer.class).getMethods()) {
          if (annotatedMethod.getJavaMember().equals("explode")) {
              method = annotatedMethod.getJavaMember();
              break;
          }
      }
      

      Now the application has the Method handle without having the "accessDeclaredMembers" permission.

      The application still cannot invoke the method as it needs to bypass security checks on access to the method first. This cannot be done without the "suppressAccessChecks" runtime permission.

      Again, remember that CDI implementations are supposed to be capable of invoking inaccessible methods and this capability is exposed through the SPI. Therefore, the application call:

      Producer<?> producer = manager.getProducerFactory(annotatedMethod, null).createProducer(null);
      

      Now, calling producer.produce() will cause the sensitive method to be invoked even though the application was not granted "accessDeclaredMembers" neither "suppressAccessChecks" permission.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  jharting Jozef Hartinger
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated: