Harmony
  1. Harmony
  2. HARMONY-1956

[classlib][luni] org.apache.harmony.luni.tests.java.lang.ClassTest.test_getClasses_subtest0() has no complete solution in JRE - the testcase should be fixed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None
    • Estimated Complexity:
      Novice

      Description

      Alexey Varlamov wrote at HARMONY-1688,
      The testcase enforces somewhat stronger limitation: AccessController.checkPermission() should never lead to nested call for SecurityManager.checkPermission(), otherwise we have recursion again. I guess if we slightly hack the environment of this testcase (without changing testing logic itself) e.g. to use custom security policy provider, we'll be able to reproduce endless recursion on RI too. The DRLVM is more vulnerable to this due to it's pure-Java ACC impl peculiarities, it fails even with the default policy.

      [...] the [...] issue has no complete solution in JRE.

      Here is the fix for the test, better handling of recursion in security machinery. This eliminates issue (2) above.
      http://issues.apache.org/jira/secure/attachment/12342647/H-1688.ClassTest.patch (ASF granted)

      See also http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM

        Issue Links

          Activity

          Hide
          Paulex Yang added a comment -

          Duplicated with HARMONY-1805.

          Show
          Paulex Yang added a comment - Duplicated with HARMONY-1805 .
          Hide
          Alexei Fedotov added a comment -

          Please close this bug as duplicate.

          Show
          Alexei Fedotov added a comment - Please close this bug as duplicate.
          Hide
          Alexey Varlamov added a comment -

          Yes, this one should be resolved duplicate. Thanks.

          Show
          Alexey Varlamov added a comment - Yes, this one should be resolved duplicate. Thanks.
          Hide
          Alexei Fedotov added a comment -

          The patches are different, but the statement is the same. Alexey V, could you please comment if these issues are truly duplicates?

          Show
          Alexei Fedotov added a comment - The patches are different, but the statement is the same. Alexey V, could you please comment if these issues are truly duplicates?

            People

            • Assignee:
              Paulex Yang
              Reporter:
              Alexei Fedotov
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development