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/Almoxarifado.php #574

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis
Vulnerabilidade encontrada:

  • O construtor da classe almoxarifado recebe um parâmetro $descricao_almoxarifado e atribui diretamente à propriedade descricao_almoxarifado, sem realizar uma validação ou sanitização dos dados. Isso pode resultar em possíveis vulnerabilidades de segurança, como injeção de código malicioso.

Para corrigir essa vulnerabilidade, é recomendado realizar a validação e sanitização dos dados recebidos no construtor. Uma abordagem comum é utilizar filtros ou regular expressions para garantir que os dados recebidos são seguros antes de atribuí-los às propriedades da classe. Além disso, a utilização de prepared statements em consultas SQL pode ajudar a prevenir ataques de injeção de SQL.

Exemplo de correção da vulnerabilidade no construtor:

public function __construct($descricao_almoxarifado)
{
    // Validar e sanitizar o parâmetro $descricao_almoxarifado
    $descricao_almoxarifado = filter_var($descricao_almoxarifado, FILTER_SANITIZE_STRING);

    $this->descricao_almoxarifado = $descricao_almoxarifado;
}

Ao aplicar a correção acima, os dados do parâmetro $descricao_almoxarifado serão validados e sanitizados antes de serem atribuídos à propriedade da classe, proporcionando uma camada adicional de segurança.

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

Feedback

  • O Comentário foi útil? 5 = Muito útil
  • O erro foi fácil de lozalicar? 5 = Muito fácil
  • Foi possível corrigir? Sim
  • O Quão fácil de corrigir? 3 = Relativamente fácil

OBS: Apesar da correção não ser complexa, ela envolveu a refatoração de outros arquivos não mencionados na issue

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