SV.STRUTS.STATIC
このエラーは、クラスが org.apache.struts.action.ActionForm を拡張し、final でない static フィードを定義する場合に報告されます。
脆弱性とリスク
final ではない static フィールドは、異なるインスタンスによって同時またはチェック時と使用時の間 (TOCTOU) に変更される可能性があるため、競合状態を生じる可能性があります。
Klocwork セキュリティ脆弱性 (SV) チェッカーは、潜在的に危険なデータを生成する呼び出しを特定します。このような呼び出しは安全でないソースと考えられます。ユーザーは攻撃者になる可能性があり、ヒューマンエラーを取り込む可能性があるため、安全でないソースはユーザーが指定した任意のデータである可能性があります。
軽減と防止
final ではない static フィールドの使用は避けます。フィールドを final または non-static にします。フィールドを final にする場合、そのフィールドを含むオブジェクトが immutable オブジェクトであることを確認します。このフィールドの値が要求に依存する場合、悪意のあるフォームが作成され、何らかの脅威にさらされる可能性があるため、このフィールドは static でないようにすることが必要です。このフィールドが要求に依存しない場合には、このフィールドを final にする必要があります。
例 1
コピー
public class SV_STRUTS_STATIC_Sample_1 extends ActionForm {
private String birthdayString;
protected Date birthday;
private static SimpleDateFormat dateConvertor;
public ActionErrors validate(ActionMapping map,
HttpServletRequest req) {
ActionErrors errors = new ActionErrors();
if (dateConvertor == null) {
dateConvertor = new SimpleDateFormat(req
.getParameter("date.format"));
}
try {
birthday = dateConvertor.parse(birthdayString);
} catch (ParseException e) {
errors.add(ActionErrors.GLOBAL_MESSAGE,
new ActionMessage("Bad date"));
}
return super.validate(map, req);
}
public String getBirthday() {
return birthdayString;
}
public void setBirthday(String birthday) {
this.birthdayString = birthday;
}
}SV.STRUTS.STATIC が 22 行目のフィールド宣言に対して報告されています。Struts: Static フォームフィールド変数 dateConvertor は final である必要があります。