You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by "Svetlana Samoilenko (JIRA)" <ji...@apache.org> on 2006/02/15 06:44:14 UTC

[jira] Updated: (HARMONY-93) some methods in java.util.Collections don't throw NPE when the parameter is null

     [ http://issues.apache.org/jira/browse/HARMONY-93?page=all ]

Svetlana Samoilenko updated HARMONY-93:
---------------------------------------

    Attachment: CollectionsTest.java

> some methods in java.util.Collections don't throw  NPE when the parameter is null
> ---------------------------------------------------------------------------------
>
>          Key: HARMONY-93
>          URL: http://issues.apache.org/jira/browse/HARMONY-93
>      Project: Harmony
>         Type: Bug
>   Components: Classlib
>     Reporter: Svetlana Samoilenko
>  Attachments: CollectionsTest.java
>
> According to j2se 1.4.2 and 1.5 specification for the Collections class  "The methods of this class all throw a NullPointerException if the collections provided to them are null".
> Harmony does not throw NPE for some methods. 
> Code to reproduce: 
> import java.util.*; 
> public class test2 { 
>     public static void main(String [] args) { 
>            SortedMap m = null;
>            Map map=null;
>            Set set=null;   
>            SortedSet sortedset=null;
>            SortedMap sortedmap =null;
>            Collection col=null;
>            try {
>                Collections.synchronizedCollection(col);
>                System.out.println("synchronizedCollection(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                Collections.synchronizedSortedMap(sortedmap);
>                System.out.println("synchronizedSortedMap(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                Collections.synchronizedMap(map); 
>                System.out.println("synchronizedMap(map) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                Collections.synchronizedSet(set); 
>                System.out.println("synchronizedSet(set) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                 Collections.synchronizedSortedSet(sortedset);
>                 System.out.println("synchronizedSortedSet(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                 Collections.unmodifiableCollection(col);
>                 System.out.println("unmodifiableCollection(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                Collections.unmodifiableMap(map);
>                System.out.println("unmodifiableMap(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                Collections.unmodifiableSet(set);
>                System.out.println("unmodifiableSet(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                 Collections.unmodifiableSortedMap(sortedmap);
>                 System.out.println("unmodifiableSortedMap(null) must throw NPE");
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }
>            try {
>                 Collections.unmodifiableSortedSet(sortedset);
>                 System.out.println("unmodifiableSortedSet(null) must throw NPE");           
>            } catch (NullPointerException e) {
>                System.out.println("PASSED");
>            }         
>     } 
>  } 
> Steps to Reproduce: 
> 1. Build Harmony (check-out on 2006-01-30) j2se subset as described in README.txt. 
> 2. Compile test2.java using BEA 1.4 javac 
> > javac -d . test2.java 
> 3. Run java using compatible VM (J9) 
> > java -showversion test2
> Output: 
> C:\tmp>C:\jrockit-j2sdk1.4.2_04\bin\java.exe -showversion test2 
> java version "1.4.2_04" 
> Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.2_04-b05) 
> BEA WebLogic JRockit(TM) 1.4.2_04 JVM (build ari-31788-20040616-1132-win-ia32, Native Threads, GC strategy: parallel) 
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> PASSED
> C:\tmp>C:\harmony\trunk\deploy\jre\bin\java -showversion test2 
> (c) Copyright 1991, 2005 The Apache Software Foundation or its licensors, as applicable. 
> synchronizedCollection(null) must throw NPE
> synchronizedSortedMap(null) must throw NPE
> synchronizedMap(map) must throw NPE
> synchronizedSet(set) must throw NPE
> synchronizedSortedSet(null) must throw NPE
> unmodifiableCollection(null) must throw NPE
> unmodifiableMap(null) must throw NPE
> unmodifiableSet(null) must throw NPE
> unmodifiableSortedMap(null) must throw NPE
> unmodifiableSortedSet(null) must throw NPE
> Suggected fix and test in attachment. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira