-
Notifications
You must be signed in to change notification settings - Fork 603
[Medium] Patch telegraf for CVE-2025-11065 #15657
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: main
Are you sure you want to change the base?
Conversation
Kanishk-Bansal
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.
|
Patch has been backported. For file, There is no There is no For file |
realsdx
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.
- Patch applies cleanly
- Security tag exists
- Release bump & Changelog
- Buddy Build
- No new ptest regression
Signed-off-by: @realsdx
Kanishk-Bansal
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.
Patch Analysis
- Buddy Build
- patch applied during the build (check
rpm.log) - patch include an upstream reference
- PR has security tag
Patch has been backported.
For file, decode_hooks.go
Many APIs are not present in the codebase. Below are the missing APIs in our version. Also, attaching the .rej file for reference.(changed the fileformat to .patch as github doesnot support .rej file to upload).
StringToURLHookFunc(), StringToNetIPAddrHookFunc(), StringToNetIPAddrPortHookFunc(), StringToNetIPPrefixHookFunc(), StringToInt8HookFunc(), StringToUint8HookFunc(), StringToInt16HookFunc(), StringToUint16HookFunc(), StringToInt32HookFunc(), StringToUint32HookFunc() StringToInt64HookFunc(), StringToUint64HookFunc(), StringToIntHookFunc(), StringToUintHookFunc(), StringToFloat32HookFunc(), StringToFloat64HookFunc(), StringToBoolHookFunc(), StringToComplex64HookFunc() StringToComplex128HookFunc()
decode_hooks.go-rej.patch
There is no decode_hooks_test.go file in our codebase.
There is noerrors.go file in our codebase but there is error.go file. So, added new APIs into this file.
For file mapstructure.go
As newDecodeError is not defined in error.go file, upstream patch failed to apply. As per the fix, err is replaced with wrapStrconvNumError(err)
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 telegraf for CVE-2025-11065
Backported: Yes (Details are given in the comment.)
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology