Skip to content

Remove unnecessary udp query#16952

Merged
omoerbeek merged 1 commit intoPowerDNS:masterfrom
jsoref:codeql-remove-stray-udp-query
Mar 5, 2026
Merged

Remove unnecessary udp query#16952
omoerbeek merged 1 commit intoPowerDNS:masterfrom
jsoref:codeql-remove-stray-udp-query

Conversation

@jsoref
Copy link
Contributor

@jsoref jsoref commented Mar 5, 2026

Short description

CodeQL Quality complained "Variable defined multiple times". This change removes the first instance which was a bad copy+paste from another test.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document, including the AI Policy, and added a "Signed-off-by" to my commits
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Copy link
Contributor

@miodvallat miodvallat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Likely correct.

@omoerbeek omoerbeek merged commit 2d43a25 into PowerDNS:master Mar 5, 2026
91 checks passed
@jsoref jsoref deleted the codeql-remove-stray-udp-query branch March 5, 2026 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants