Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Análise de vulnerabilidade do arquivo html/atendido/profile_familiar.php #342

Closed
joaopontes22 opened this issue May 10, 2024 · 1 comment
Labels
Security MN análise

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis
A primeira vulnerabilidade no código é a presença da função extract($_REQUEST), que pode causar problemas de segurança, especialmente quando há uma mistura de dados GET, POST e COOKIES. Isso pode resultar em vulnerabilidades de segurança, como a possibilidade de sobrescrita de variáveis.

Além disso, existe uma vulnerabilidade de Injeção de SQL na linha $dependente = $pdo->query("SELECT *, ap.parentesco AS parentesco FROM atendido_familiares af LEFT JOIN pessoa p ON p.id_pessoa = af.pessoa_id_pessoa LEFT JOIN atendido_parentesco ap ON ap.idatendido_parentesco = af.atendido_parentesco_idatendido_parentesco WHERE af.idatendido_familiares = " . $_GET['id_dependente'] ?? null);. A concatenação direta de valores do $_GET em uma consulta SQL permite que um invasor execute uma injeção de SQL.

Recomenda-se usar prepared statements para evitar esse tipo de vulnerabilidade, para que os valores dos parâmetros da consulta sejam tratados separadamente dos comandos SQL.

A terceira vulnerabilidade é a presença de scripts em excesso, como múltiplos links para arquivos JavaScript e CSS. Isso pode aumentar o tempo de carregamento da página e dificultar a manutenção do código.

Por fim, a ausência de validação de entrada nos campos de formulário (como campos de CPF, data de nascimento, etc.) pode permitir que os usuários insiram dados inválidos ou maliciosos. Recomenda-se validar e sanitizar todos os dados de entrada antes de processá-los no código.

@joaopontes22 joaopontes22 added the Security MN análise label May 10, 2024
nicolly015 added a commit that referenced this issue Sep 19, 2024
@nicolly015
Copy link
Contributor

  • O comentário foi útil? 4 = Útil

  • O erro foi fácil de localizar? 5 = Muito fácil

  • Foi possível corrigir o erro? Possível

  • O quão fácil de corrigir? 5 = Muito fácil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Security MN análise
Projects
None yet
Development

No branches or pull requests

2 participants