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/controller/deletar_diretorio_tabelas.php #214

Open joaopontes22 opened 5 months ago

joaopontes22 commented 5 months ago

Vulnerability Analysis Há uma vulnerabilidade devido ao uso do scandir sem a devida validação dos arquivos/diretórios retornados pela função. Isso pode permitir a um usuário mal-intencionado modificar o nome de um diretório/arquivo para incluir uma referência ".." e assim acessar diretórios acima do especificado.

Para corrigir essa vulnerabilidade, é recomendável adicionar uma validação para garantir que os arquivos retornados pela função scandir sejam válidos dentro do diretório especificado. Isso pode ser feito verificando se o nome do arquivo começa com "." antes de continuar com a exclusão.

Além disso, é importante considerar a segurança ao excluir arquivos e diretórios, garantindo que apenas os arquivos/diretórios desejados sejam excluídos e evitando a exclusão acidental de dados importantes. Uma estratégia para mitigar esse tipo de risco é garantir que o diretório passado como parâmetro para a função delTree seja válido e restrito às operações permitidas.

Uma ação adicional a ser considerada é adicionar permissões de acesso e restrições de arquivo/diretório para restringir as operações que podem ser realizadas pela função delTree, a fim de evitar manipulações maliciosas por parte de usuários mal-intencionados.

nicolly015 commented 4 months ago

Pesquisa feita: