Solidity Bad Practices: Default Function Visibility

Una función no especifica un modificador de acceso explícito.

Al desarrollar un contrato inteligente de Solidity, los desarrolladores pueden configurar el modificador de acceso de una función para controlar quién puede invocarla. Si un desarrollador no especifica un modificador de acceso, la función se establece por defecto en pública, lo que permite que un actor malicioso llame a la función sin autorización.

Ejemplo 1: El siguiente código no configura el modificador de acceso en ambas funciones y, por lo tanto, cualquiera puede invocarlas, lo que permite a un usuario eludir la llamada a require.

function withdrawWinnings() {
    require(uint32(msg.sender) == 0);
    _sendWinnings();
}

function _sendWinnings() {
    msg.sender.transfer(this.balance);
}