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

        Hide
        Vladimir Ivanov added a comment -

        patch

        Show
        Vladimir Ivanov added a comment - patch
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development