Revert commit 3505c4b

The fix did not address the issue. It only made the constructor not
fail with tests succeeding due to setter binding instead.

See gh-34043

Author: rstoyanchev

Diff for: spring-context/src/main/java/org/springframework/validation/DataBinder.java

@@ -1082,7 +1082,7 @@ private <V> Map<String, V> createMap(
 			}
 			int startIdx = paramPath.length() + 1;
 			int endIdx = name.indexOf(']', startIdx);
-			String nestedPath = ((name.length() > endIdx + 1) ? name.substring(0, endIdx + 2) : "");
+			String nestedPath = name.substring(0, endIdx + 2);
 			boolean quoted = (endIdx - startIdx > 2 && name.charAt(startIdx) == '\'' && name.charAt(endIdx - 1) == '\'');
 			String key = (quoted ? name.substring(startIdx + 1, endIdx - 1) : name.substring(startIdx, endIdx));
 			if (map == null) {
@@ -1116,7 +1116,7 @@ private static SortedSet<Integer> getIndexes(String paramPath, ValueResolver val
 		SortedSet<Integer> indexes = null;
 		for (String name : valueResolver.getNames()) {
 			if (name.startsWith(paramPath + "[")) {
-				int endIndex = name.indexOf(']', paramPath.length() + 1);
+				int endIndex = name.indexOf(']', paramPath.length() + 2);
 				String rawIndex = name.substring(paramPath.length() + 1, endIndex);
 				int index = Integer.parseInt(rawIndex);
 				indexes = (indexes != null ? indexes : new TreeSet<>());