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/saude/enfermidade_upload.php #241

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis
Vulnerabilidades encontradas no código:

  1. XSS (Cross-Site Scripting) na linha 18:

    • A variável $id_fichamedica não está sendo sanitizada antes de ser utilizada no redirecionamento do header. Um atacante poderia manipular o valor desta variável para injetar scripts maliciosos, levando a um ataque de XSS.
  2. Injeção de SQL na linha 9:

    • Os valores fornecidos pelo usuário em $_REQUEST não estão sendo filtrados ou validados antes de serem utilizados em uma declaração SQL. Isso deixa a aplicação vulnerável a ataques de injeção de SQL.
  3. Manipulação de sessão na linha 6:

    • A função extract($_REQUEST) está sendo utilizada para extrair os dados do usuário, o que pode causar ambiguidade e vulnerabilidades de segurança. É recomendado não usar extract e acessar diretamente os valores do array $_REQUEST.
  4. XSS potencial na linha 27:

    • A variável $e, que contém a mensagem de erro do PDOException, não está sendo sanitizada antes de ser exibida na tela. Isso pode ser utilizado por um atacante para realizar um ataque de XSS.

Recomendações de correção:

  • Utilize prepared statements com bindValue para evitar a injeção de SQL.
  • Sanitize/validate todos os dados fornecidos pelo usuário antes de utilizá-los.
  • Evite usar extract e acesse diretamente os valores do array $_REQUEST.
  • Para exibir mensagens de erro, utilize htmlspecialchars para evitar XSS.

Corrigir essas vulnerabilidades ajudará a garantir a segurança e a integridade da sua aplicação.

@joaopontes22 joaopontes22 added the Security MN análise label May 9, 2024
FlameOff added a commit that referenced this issue Oct 25, 2024
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