Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

FIX: SQLi in poller's broker configuration #11760

Merged
merged 2 commits into from
Sep 16, 2022

Conversation

hyahiaoui-ext
Copy link
Contributor

@hyahiaoui-ext hyahiaoui-ext commented Sep 14, 2022

Description

Sanitizing pollers broker config queries.

Fixes # MON-14981

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Target serie

  • 21.04.x
  • 21.10.x
  • 22.04.x
  • 22.10.x (master)

Checklist

Community contributors & Centreon team

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have rebased my development branch on the base branch (master, maintenance).

a-launois
a-launois previously approved these changes Sep 14, 2022
@sonarqube-decoration
Copy link

@hyahiaoui-ext hyahiaoui-ext merged commit 323f291 into develop Sep 16, 2022
@hyahiaoui-ext hyahiaoui-ext deleted the MON-14981-sqli-pollers-broker-configuration branch September 16, 2022 08:28
hyahiaoui-ext added a commit that referenced this pull request Sep 16, 2022
* sanitize and bind pollers broker config queries

* applying suggested changes
hyahiaoui-ext added a commit that referenced this pull request Sep 16, 2022
* sanitize and bind pollers broker config queries

* applying suggested changes
hyahiaoui-ext added a commit that referenced this pull request Sep 16, 2022
* sanitize and bind pollers broker config queries

* applying suggested changes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants