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/pet/profile_pet.php #257

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis

  1. Vulnerabilidade de Injeção de SQL: A função addTipoExame() no código parece estar suscetível a uma injeção de SQL. Se o valor tipo_exame for diretamente concatenado na query SQL sem nenhuma validação adequada, isso pode permitir que um usuário mal-intencionado execute um ataque de injeção de SQL. É recomendado que você use uma abordagem parametrizada ao construir suas consultas SQL.

  2. Uso de Funções e Dependências Não Utilizadas: Existem várias importações de scripts e bibliotecas que parecem não ser usadas no código, como onlyNumbers.js, onlyChars.js, mascara.js, lista.js, printThis.js, ckeditor.js, entre outros. Certifique-se de remover qualquer dependência ou script que não esteja sendo utilizado para reduzir a complexidade e melhorar o desempenho do código.

  3. Repetição de Código: Há repetição de blocos de código que importam os mesmos estilos, scripts e arquivos de ícones várias vezes no código. Isso pode ser otimizado agrupando todas as importações iguais em um único bloco, a fim de deixar o código mais limpo e manutenível.

  4. Controle de Acesso e Permissões: A verificação de permissão no código parece estar verificando apenas se o usuário possui permissão, mas não está implementando nenhuma lógica adicional com base nessa permissão. É importante garantir que a verificação de permissão seja segura e abranja todas as funcionalidades que possam estar sujeitas a restrições de acesso.

  5. Manipulação de Sessão: É recomendável adicionar medidas de segurança adicionais à manipulação de sessões, como verificar a integridade dos dados da sessão, limitar o tempo de expiração da sessão e garantir que as informações sensíveis da sessão sejam armazenadas de forma segura.

Certifique-se de abordar essas vulnerabilidades e melhorias no código para garantir a segurança e eficiência do seu sistema.

@joaopontes22 joaopontes22 added the Security MN análise label May 9, 2024
angel-jgpv added a commit that referenced this issue Sep 12, 2024
@angel-jgpv
Copy link
Contributor

Resolvido By: Lenart
12/09/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