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 である必要があります。