Weak SecurityManager Check: Overridable Method

セキュリティチェックを実行する最終ではない (final ではない) メソッドは、上書きされる可能性があるため、セキュリティチェックが省略される可能性があります。

メソッドが子クラスにより上書きされる場合、子クラスは親クラスでセキュリティチェックを省略する場合があります。
例 1: 次のコード doSecurityCheck() は、セキュリティチェックを実行しますが、子クラスにより上書きされる可能性があります。

public class BadSecurityCheck {
    private int id;

    public BadSecurityCheck() {
        doSecurityCheck();
        id = 1;
    }
    protected void doSecurityCheck() {
        SecurityManager sm = System.getSecurityManager();
        if (sm != null) {
            sm.checkPermission(new SomePermission("SomeAction"));
        }
    }
}

この例では、SecurityManager 権限が付与されていない場合、SecurityException 例外がスローされます。これはランタイム例外で、プログラムの実行は停止されます。BadSecurityCheck は final でなく、doSecurityCheck() メソッドは protected で final でないため、このクラスをサブクラス化して関数をオーバーライドすることができます。

例 2: 次のコードでは、サブクラスによって doSecurityCheck() がオーバーライドされています。

public class EvilSubclass extends BadSecurityCheck {
    private int id;

    public EvilSubclass() {
        super();
    }
    protected void doSecurityCheck() {
        //do nothing
    }
}

EvilSubclass をインスタンス化すると、最初にコンストラクタが super() でスーパークラスのコンストラクタを呼び出します。その結果、関数 doSecurityCheck() が呼ばれますが、Java は最初にサブクラス内の関数を探してからスーパークラスを探すため、セキュリティ チェックを回避する攻撃者のメソッドが呼び出され、id が 1 に設定されます。