nilsonLazarin / WeGIA

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

Análise de vulnerabilidade do arquivo html/socio/sistema/editar_socio.php #176

Open joaopontes22 opened 6 months ago

joaopontes22 commented 6 months ago

Vulnerability Analysis Vulnerabilidades encontradas:

  1. SQL Injection: A vulnerabilidade está presente nas consultas SQL sem a parametrização correta dos inputs do usuário. Por exemplo, na linha 8, a variável $id_pessoa é concatenada diretamente na query SQL, o que pode permitir um ataque de injeção SQL se o valor contido nessa variável for controlado por um usuário mal-intencionado.

  2. Path Traversal: As linhas 12 a 22 apresentam uma vulnerabilidade de Path Traversal ao incluírem arquivos sem verificar se eles estão localizados em diretórios seguros. Essa prática pode levar a problemas de segurança, permitindo que um invasor acesse arquivos sensíveis no servidor.

  3. Repetição de Código: Há repetições de código nas verificações de permissões e redirecionamentos de mensagens de erro para o mesmo trecho do código. Essa repetição pode dificultar a manutenção do código e aumentar a possibilidade de erros.

Explicação:

  1. Para prevenir a vulnerabilidade de SQL Injection, é recomendável utilizar prepared statements ou funções de escape de strings, como mysqli_real_escape_string, para evitar a inserção de código malicioso nas consultas SQL.

  2. Para impedir ataques de Path Traversal, é importante validar os diretórios de inclusão de arquivos e limitar o acesso somente aos arquivos e diretórios permitidos.

  3. Para melhorar a manutenção e legibilidade do código, é aconselhável centralizar o código de redirecionamento e mensagens de erro em funções reutilizáveis, evitando repetições desnecessárias de código.

Além disso, é importante considerar a organização e manutenção do código, bem como a adoção de práticas de segurança recomendadas para garantir a proteção adequada do sistema contra vulnerabilidades e ataques.

nicolly015 commented 3 months ago