Index: modules/beans/make/common/build.xml =================================================================== --- modules/beans/make/common/build.xml.orig 2006-03-21 20:49:57.000000000 +0000 +++ modules/beans/make/common/build.xml 2006-03-22 08:20:01.000000000 +0000 @@ -96,7 +96,6 @@ - Index: modules/beans/src/main/java/java/beans/PropertyChangeSupport.java =================================================================== --- modules/beans/src/main/java/java/beans/PropertyChangeSupport.java.orig 2006-03-21 07:48:18.000000000 +0000 +++ modules/beans/src/main/java/java/beans/PropertyChangeSupport.java 2006-03-22 08:18:50.000000000 +0000 @@ -52,6 +52,8 @@ * @com.intel.drl.spec_ref */ public PropertyChangeSupport(Object sourceBean) { + if(sourceBean == null) + throw new NullPointerException(); this.sourceBean = sourceBean; } Index: modules/beans/src/test/java/org/apache/harmony/tests/beans/PropertyChangeSupportTest.java =================================================================== --- modules/beans/src/test/java/org/apache/harmony/tests/beans/PropertyChangeSupportTest.java.orig 2006-03-21 07:48:16.000000000 +0000 +++ modules/beans/src/test/java/org/apache/harmony/tests/beans/PropertyChangeSupportTest.java 2006-03-22 08:20:12.000000000 +0000 @@ -79,6 +79,15 @@ PropertyChangeListener[] pcls = readPropertyChangeListeners(); } + public void testNullPointerExpection() { + try { + PropertyChangeSupport bd = new PropertyChangeSupport(null); + fail("Constructor PropertyChangeSupport(null) should "+ + "throw NullPointerException"); + } catch (NullPointerException e) { + } + } + /** * */