Index: modules/beans/make/common/build.xml =================================================================== --- modules/beans/make/common/build.xml.orig 2006-03-22 08:19:25.000000000 +0000 +++ modules/beans/make/common/build.xml 2006-03-22 08:19:39.000000000 +0000 @@ -97,7 +97,6 @@ - Index: modules/beans/src/main/java/java/beans/VetoableChangeSupport.java =================================================================== --- modules/beans/src/main/java/java/beans/VetoableChangeSupport.java.orig 2006-03-21 07:48:18.000000000 +0000 +++ modules/beans/src/main/java/java/beans/VetoableChangeSupport.java 2006-03-22 08:19:01.000000000 +0000 @@ -51,6 +51,8 @@ * @com.intel.drl.spec_ref */ public VetoableChangeSupport(Object sourceBean) { + if(sourceBean == null) + throw new NullPointerException(); this.sourceBean = sourceBean; } Index: modules/beans/src/test/java/org/apache/harmony/tests/beans/VetoableChangeSupportTest.java =================================================================== --- modules/beans/src/test/java/org/apache/harmony/tests/beans/VetoableChangeSupportTest.java.orig 2006-03-21 07:48:16.000000000 +0000 +++ modules/beans/src/test/java/org/apache/harmony/tests/beans/VetoableChangeSupportTest.java 2006-03-22 08:17:35.000000000 +0000 @@ -118,6 +118,15 @@ } } + public void testNullPointerExpection() { + try { + VetoableChangeSupport bd = new VetoableChangeSupport(null); + fail("Constructor VetoableChangeSupport(null) should "+ + "throw NullPointerException"); + } catch (NullPointerException e) { + } + } + /** * */