java 安全数组_java – 安全 – 数组直接存储

我的代码是—>

public final void setSelectedObjectsList(final ScheduleDTO[] selectedObjectsList)

// Security - Array is stored directly

//The user-supplied array 'selectedObjectsList' is stored directly.

{

if (selectedObjectsList != null) {

this.selectedObjectsList = selectedObjectsList.clone();

} else {

this.selectedObjectsList = null;

}

}

这已经在处理防御性复制,为什么声纳会在功能参数上对我大吼大叫.

再次,谢谢你的呐喊和时间.

解决方法:

不确定Sonar在想什么,但使用clone()进行防御性浅层复制应该可以正常使用数组,就像Arrays.copyOf和System.arrayCopy()一样.

另一方面,由于您已经在调用数组列表：selectedObjectsList,您还可以将其设为实际列表并重构一下：

public final void setSelectedSchedules(List selectedSchedules) {

this.selectedSchedules = selectedSchedules != null ? new ArrayList(selectedSchedules) : null;

}

标签：java,findbugs,sonarqube

来源： https://codeday.me/bug/20190529/1178085.html