Harmony
  1. Harmony
  2. HARMONY-1229

[classlib][auth] compatibility: new SubjectDomainCombiner().combine( {null}, ...) throws NPE on Harmony and works silently on RI

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.0M2, 5.0M14
    • Component/s: Classlib
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      The method javax.security.auth.SubjectDomainCombiner.combine(ProtectionDomain[], new ProtectionDomain[]) throws NPE on Harmony for 'null' entry in the domain-array while RI works silently.

      ================== test.java ===================
      import java.security.ProtectionDomain;
      import javax.security.auth.SubjectDomainCombiner;
      import javax.security.auth.Subject;

      public class test {

      public static void main (String[] args) {
      System.out.println("res= " + new SubjectDomainCombiner(new Subject()).combine(new ProtectionDomain[]

      {null}, new ProtectionDomain[] {null}

      ));
      }
      }
      ============================================

      Output:
      C:\tmp\tmp17>C:\jrockit-jdk1.5.0-windows-ia32\bin\java.exe -cp . -showversion test
      java version "1.5.0"
      Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0-b64)
      BEA WebLogic JRockit(R) (build dra-38972-20041208-2001-win-ia32, R25.0.0-75, GC: System optimized over throughput (initial strategy singleparpar))

      res= [Ljava.security.ProtectionDomain;@1b6436

      C:\tmp\tmp17>C:\harmony\classlib1.5\deploy\jdk\jre\bin\java.exe -cp . -showversion test
      java version 1.5 (subset)

      (c) Copyright 1991, 2006 The Apache Software Foundation or its licensors, as applicable.
      Exception in thread "main" java.lang.NullPointerException
      at javax.security.auth.SubjectDomainCombiner.combine(SubjectDomainCombiner.java:95)
      at test.main(test.java:8)

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1310d 8h 40m 1 Oliver Deakin 23/Mar/10 13:42
        Oliver Deakin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 6.0M2 [ 12314804 ]
        Fix Version/s 5.0M14 [ 12314803 ]
        Resolution Fixed [ 1 ]
        Hide
        Oliver Deakin added a comment -

        Thanks Vladimir! Fix and regression test applied at repo revision r926569. Please check they were applied as expected.

        Show
        Oliver Deakin added a comment - Thanks Vladimir! Fix and regression test applied at repo revision r926569. Please check they were applied as expected.
        Oliver Deakin made changes -
        Assignee Stepan Mishura [ smm ] Oliver Deakin [ odeakin ]
        Alexey Petrenko made changes -
        Patch Info [Patch Available]
        Stepan Mishura made changes -
        Assignee Stepan Mishura [ smm ]
        Vladimir Ivanov made changes -
        Field Original Value New Value
        Attachment SubjectDomainCombiner.patch [ 12339199 ]
        Hide
        Vladimir Ivanov added a comment -

        patch

        Show
        Vladimir Ivanov added a comment - patch
        Vladimir Ivanov created issue -

          People

          • Assignee:
            Oliver Deakin
            Reporter:
            Vladimir Ivanov
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development