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 classes/Beneficios.php #566

Closed
joaopontes22 opened this issue May 29, 2024 · 1 comment
Closed

Análise de vulnerabilidade do arquivo classes/Beneficios.php #566

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis
Vulnerabilidades encontradas no código:

  1. Falta de restrição de acesso aos atributos e métodos: Os atributos $id_beneficios e $descricao_beneficios, bem como os métodos getId_beneficios(), getDescricao_beneficios(), setId_beneficios() e setDescricao_beneficios(), estão definidos como private. Isso significa que apenas a própria classe Beneficios pode acessá-los diretamente. Porém, se outras classes precisarem acessar ou modificar esses atributos, será necessário alterar o modificador de acesso para protected ou public.

  2. Falta de validações nos métodos setId_beneficios() e setDescricao_beneficios(): Atualmente, esses métodos simplesmente atribuem os valores dos parâmetros aos atributos correspondentes, sem realizar qualquer tipo de validação. É importante adicionar validações, como por exemplo verificar se os valores passados são do tipo esperado, se estão dentro de um intervalo válido, entre outras verições necessárias de acordo com a lógica de negócio.

  3. Dependência direta da classe Beneficiados: A classe Beneficios está estendendo diretamente a classe Beneficiados. Isso pode causar um acoplamento forte entre essas duas classes, o que pode dificultar a manutenção e a reutilização do código. É necessário avaliar se essa herança é realmente necessária ou se seria melhor usar composição ou outro método para injetar a dependência de Beneficiados em Beneficios.

Corrigindo essas vulnerabilidades, o código ficaria mais seguro e aderente às boas práticas de programação.

@joaopontes22 joaopontes22 added the Security MN análise label May 29, 2024
@GabrielPintoSouza GabrielPintoSouza self-assigned this Jun 17, 2024
GabrielPintoSouza added a commit that referenced this issue Jun 17, 2024
@GabrielPintoSouza
Copy link
Collaborator

Feedback

  • O comentário foi útil? 4 = Útil
  • O erro foi fácil de localizar? 5 = Muito fácil
  • Foi possível corrigir? 5 = Sim
  • O quão fácil de corrigir? 5 = Muito 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