Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8272608

java_lang_System::allow_security_manager() doesn't set its initialization flag

    XMLWordPrintable

Details

    • b09
    • 17
    • b15

    Backports

      Description

        javaClasses.cpp:

        bool java_lang_System::allow_security_manager() {
          static int initialized = false;
          static bool allowed = true; // default
          if (!initialized) {
            oop base = vmClasses::System_klass()->static_field_base_raw();
            int never = base->int_field(_static_never_offset);
            allowed = (base->int_field(_static_allow_security_offset) != never);
          }
          return allowed;
        }

        We never set `initialized` to true, so every call to allow_security_manager() re-reads the System class field.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8296522 java_lang_System::allow_security_manager() doesn't set its initialization flag

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8298483 java_lang_System::allow_security_manager() doesn't set its initialization flag

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Enhancement - null JDK-8195744 Avoid calling ClassLoader.checkPackageAccess if security manager is not installed

            • P3 - Major loss of function.
            • Resolved
            links to

            Commit Commit openjdk/jdk17u-dev/a61b74e2

            Commit Commit openjdk/jdk/08f0be83

            Review Review openjdk/jdk17u-dev/843

            Review Review openjdk/jdk/5268

            Activity

              People

                msaavedrasil Matias Saavedra Silva (Inactive)
                dholmes David Holmes
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: