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/pessoa/editar_documentacao.php #269

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis

  1. SQL Injection: O código está vulnerável a ataques de injeção de SQL devido ao uso direto das variáveis passadas pelo usuário ($_POST) na construção da query SQL. Isso pode permitir que um atacante execute comandos maliciosos no banco de dados. Recomenda-se o uso de prepared statements para evitar essa vulnerabilidade.

  2. XSS (Cross-Site Scripting): O código está vulnerável a ataques de XSS devido ao retorno da variável $_GET na resposta JSON. Se a variável $_GET['sql'] contiver scripts maliciosos, eles serão executados no navegador do usuário. É importante sanitizar e/ou escapar os dados antes de exibi-los no front-end para evitar esse tipo de ataque.

  3. Exposição de Informações Sensíveis: A exibição da query SQL no retorno da requisição pode expor informações sensíveis do banco de dados, como a estrutura da tabela e os dados armazenados. Isso pode facilitar potenciais ataques por parte de hackers. Recomenda-se não retornar informações sensíveis desse tipo nas respostas da aplicação.

Para corrigir essas vulnerabilidades, é recomendável:

  • Utilizar prepared statements para executar consultas SQL com segurança.
  • Sanitizar e/ou escapar os dados antes de exibi-los no front-end para prevenir XSS.
  • Evitar retornar informações sensíveis, como queries SQL, nas respostas da aplicação, especialmente para usuários não autenticados.
@joaopontes22 joaopontes22 added the Security MN análise label May 9, 2024
nicolly015 added a commit that referenced this issue Sep 4, 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? 4 = 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