Weak SecurityManager Check: Overridable Method

Los métodos no finales que realizan comprobaciones de seguridad se pueden anular de forma que se eludan las comprobaciones de seguridad.

Si una clase secundaria reemplaza un método, la clase secundaria puede reemplazar las comprobaciones de seguridad en la clase principal.
Ejemplo 1: en el código siguiente, doSecurityCheck() realiza una comprobación de seguridad y se puede reemplazar con una clase secundaria.

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"));
        }
    }
}

En este ejemplo, si el permiso SecurityManager no está autorizado, se lanzará una excepción SecurityException, que es una excepción de tiempo de ejecución y detendrá la ejecución del programa. Como BadSecurityCheck no es final y el método doSecurityCheck() está protected y no es final, significa que esta clase se puede dividir en subclases para anular esta función.

Ejemplo 2: en el siguiente código, doSecurityCheck() queda anulado por una subclase:

public class EvilSubclass extends BadSecurityCheck {
    private int id;

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

Cuando se instancia EvilSubclass, el constructor primero llama a super(), para invocar al constructor de la súperclase. Este a su vez llama a la función doSecurityCheck(), pero Java buscará primero la función en la subclase antes de buscar en la superclase, por tanto invocando el método controlado por el usuario malintencionado que elude la comprobación de seguridad, por lo que id se establecerá en 1.