Uploaded image for project: 'Harmony'
  1. Harmony
  2. HARMONY-855

[Classlib][util]Some java.uiti.Collections methods do not throw unspecified NPE while RI does

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None
    • Environment:
      Windows XP professional
    • Estimated Complexity:
      Novice

      Description

      Build: classlib+VM revision 421248 msvc debug

      Problem:

      Some Collections.checked... methods throw NPE if one of their arguments is null in RI but don't throw in Harmony

      Code to reproduce:

      import java.util.*;

      public class Test {
      public static void main (String[] args) {
      try

      { Collections.checkedSortedMap(new TreeMap(), null, Short.TYPE); System.err.println("fail 1"); }

      catch (NullPointerException e) {
      }
      try

      { Collections.checkedSortedMap(new TreeMap(), Float.TYPE, null); System.err.println("fail 1.1"); }

      catch (NullPointerException e) {
      }
      try

      { Collections.checkedMap(new TreeMap(), null, Float.TYPE); System.err.println("fail 2"); }

      catch (NullPointerException e) {
      }
      try

      { Collections.checkedMap(new TreeMap(), Float.TYPE, null); System.err.println("fail 2.2"); }

      catch (NullPointerException e) {
      }
      try

      { Collections.checkedCollection(new Stack(), null); System.err.println("fail 3"); }

      catch (NullPointerException e) {
      }
      }
      }

      in RI this test prints nothing, in Harmony all 'fail' messages are printed

      Please see the patch attached. It contains:

      • JUnit test for the problem
      • patch for java.util.Collections

        Attachments

        1. harmony-855.diff
          4 kB
          spark shen
        2. patch.txt
          4 kB
          Anton Luht

          Activity

            People

            • Assignee:
              paulex Paulex Yang
              Reporter:
              aluht Anton Luht
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: