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/saude/statusUpdate.php #223

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

Comments

@joaopontes22
Copy link
Contributor

Vulnerability Analysis

  1. Injeção de SQL: O código está vulnerável a ataques de injeção de SQL devido à forma como a consulta SQL está sendo construída. O valor de $saude_medicacao_status_idsaude_medicacao_status é diretamente concatenado na consulta sem nenhum tipo de preparação ou sanitização, o que pode permitir que um atacante injete comandos maliciosos no banco de dados.

Para corrigir esta vulnerabilidade, você deve usar prepared statements para garantir a segurança e a integridade dos dados. Aqui está um exemplo de como você pode modificar o código para evitar a injeção de SQL:

$prep = $pdo->prepare("UPDATE saude_medicacao SET saude_medicacao_status_idsaude_medicacao_status = :status WHERE id_medicacao = :id_medicacao");
$prep->bindParam(':status', $saude_medicacao_status_idsaude_medicacao_status, PDO::PARAM_INT);
$prep->bindParam(':id_medicacao', $id_medicacao, PDO::PARAM_INT);
$prep->execute();

Certifique-se de substituir os valores associados às declarações de bindParam pelos valores corretos que você deseja inserir no banco de dados.

Além da vulnerabilidade de injeção de SQL, também é importante validar todos os dados de entrada do usuário para evitar possíveis ataques e garantir a segurança da aplicação.

@joaopontes22 joaopontes22 added the Security MN análise label May 8, 2024
@FlameOff
Copy link
Contributor

Substituição feita, variáveis agora tem seus bindParam para fazer o envio das informações para saber os status

FlameOff added a commit that referenced this issue Sep 13, 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