[BEANUTILS-368] NullPointerException in BeanUtilsBean .setProperty(), line: 930

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.8.1
• Fix Version/s: 1.8.2
• Component/s: Bean / Property Utils

Environment

JDK 1.6

• Assignee: Niall Pemberton
• Reporter: Peter Fassev
• Created: 09/Nov/09 11:28
• Updated: 25/Sep/16 22:17
• Resolved: 09/Nov/09 12:30

Description

There is a NullPointerException in the method BeanUtilsBeans.setProperty(Object bean, String name, Object value) method:

java.lang.NullPointerException
at org.apache.commons.beanutils.BeanUtilsBean.setProperty(BeanUtilsBean.java:930)

The problem is, that few rows earlear the target may be null:

target = getPropertyUtils().getProperty(target, resolver.next(name)); // NOTE, the result may be NULL.

In such case, the condition at line 930

} else if (target.getClass().isArray() && index >= 0) {

will race a NullPointerException. In the old 1.7.0 version there has been no such condition or has been executed later...

Proposed FIX: Please add at new line, which checks the "target"

target = getPropertyUtils().getProperty(target, resolver.next(name)); // NOTE, the result may be NULL.
if (target == null)

{ return; }

NOTE. Such check may be added also to the other similar methods, like BeanUtilsBean.copyProperty().

Regards
Peter

Issue Links

• relates to: BEANUTILS-345 BeanUtilsBean.setProperty does not handle some kind of nested properties