-
Notifications
You must be signed in to change notification settings - Fork 603
[Medium] Patch coredns for CVE-2025-68151 #15515
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
base: 3.0-dev
Are you sure you want to change the base?
Conversation
|
Buddy Build has passed ! |
suresh-thelkar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems all the essential security features from the upstream patch are correctly included:
- gRPC connection and stream limiting (
MaxGRPCStreams,MaxGRPCConnections) - HTTPS connection limiting (
MaxHTTPSConnections) netutil.LimitListenerusage for connection limiting- Default limits: 256 max streams, 200 max connections
- New plugins:
grpc_serverandhttps
Since the number of code changes are huge, It would be better if we can also run the full build and proceed further.
|
Full build has passed ! |
|
Waiting for full build results. |
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
Patch coredns for CVE-2025-68151
Upstream patch has been backported manually.
Patch for
https3module has not been included as this module does not exist in our version of coredns.vendor/golang.org/x/net/netutil/listen.gohas been added to vendor directory and updated "module.txt" for the same.Upstream Patch Reference: https://github.com/coredns/coredns/commit/0d8cbb1a6bcb6bc9c1a489865278b8725fa20812.patch
Change Log
Does this affect the toolchain?
NO
Links to CVEs
Test Methodology
Local build was successful.
License check script shows no warning.
Build log
coredns-1.11.4-13.azl3.src.rpm.log
coredns-1.11.4-13.azl3.src.rpm.test.log
Patch applies cleanly