Exclude any Java library type from nested constructor binding

rstoyanchev · rstoyanchev · commit d59b2924d3ca · 2023-12-01T17:04:49.000Z
Closes gh-31709
diff --git a/spring-context/src/main/java/org/springframework/validation/DataBinder.java b/spring-context/src/main/java/org/springframework/validation/DataBinder.java
@@ -1018,7 +1018,8 @@ private Object createObject(ResolvableType objectType, String nestedPath, ValueR
 	protected boolean shouldConstructArgument(MethodParameter param) {
 		Class<?> type = param.nestedIfOptional().getNestedParameterType();
 		return !(BeanUtils.isSimpleValueType(type) ||
-				Collection.class.isAssignableFrom(type) || Map.class.isAssignableFrom(type) || type.isArray());
+				Collection.class.isAssignableFrom(type) || Map.class.isAssignableFrom(type) || type.isArray() ||
+				type.getPackageName().startsWith("java."));
 	}
 
 	private void validateConstructorArgument(

Original file line number	Diff line number	Diff line change
`@@ -1018,7 +1018,8 @@ private Object createObject(ResolvableType objectType, String nestedPath, ValueR`
`1018`	`1018`	`protected boolean shouldConstructArgument(MethodParameter param) {`
`1019`	`1019`	`Class<?> type = param.nestedIfOptional().getNestedParameterType();`
`1020`	`1020`	`return !(BeanUtils.isSimpleValueType(type) \|\|`
`1021`		`- Collection.class.isAssignableFrom(type) \|\| Map.class.isAssignableFrom(type) \|\| type.isArray());`
	`1021`	`+ Collection.class.isAssignableFrom(type) \|\| Map.class.isAssignableFrom(type) \|\| type.isArray() \|\|`
	`1022`	`+ type.getPackageName().startsWith("java."));`
`1022`	`1023`	`}`
`1023`	`1024`
`1024`	`1025`	`private void validateConstructorArgument(`