nilsonLazarin / WeGIA

WeGIA: Web gerenciador para instituições assistenciais
Other
5 stars 6 forks source link

Análise de vulnerabilidade do arquivo html/socio/configuracao/index.php #219

Open joaopontes22 opened 2 months ago

joaopontes22 commented 2 months ago

Vulnerability Analysis

  1. A principal vulnerabilidade presente no código é a exposição de informações sensíveis, como a senha do usuário administrador. A linha $senha = $dados_admin['senha_d']; recupera a senha do usuário admin do banco de dados e a exibe diretamente no formulário, tanto no campo de senha quanto no campo de confirmação de senha. Isso é extremamente perigoso, pois expõe a senha em texto simples na página e pode ser acessada por qualquer pessoa que visualize o código-fonte ou inspecione o formulário.

  2. Outra vulnerabilidade potencial é a injeção de SQL nos comandos de consulta ao banco de dados. Por exemplo, na linha $dados_config = mysqli_fetch_array(mysqli_query($conexao, "select * from configuracao where id=1"));, a consulta está sendo feita diretamente concatenando o número 1 na consulta. Isso abre a possibilidade de um atacante manipular a URL e explorar a vulnerabilidade de injeção de SQL.

  3. O código também não está tratando de forma adequada a validação dos dados do formulário, o que pode levar a vulnerabilidades de cross-site scripting (XSS) se as entradas não forem devidamente sanitizadas.

Para corrigir essas vulnerabilidades, é importante:

nicolly015 commented 2 months ago