Skip to content

Security: Fix CVE-2026-32282 (Go stdlib os.Root.Chmod) SRVKP-11615#2880

Merged
tekton-robot merged 1 commit into
release-v0.37.6from
fix/SRVKP-11615-cve-2026-32282-go-stdlib-release-v0.37.6-attempt-2
May 20, 2026
Merged

Security: Fix CVE-2026-32282 (Go stdlib os.Root.Chmod) SRVKP-11615#2880
tekton-robot merged 1 commit into
release-v0.37.6from
fix/SRVKP-11615-cve-2026-32282-go-stdlib-release-v0.37.6-attempt-2

Conversation

@divyansh42
Copy link
Copy Markdown
Member

Summary

This PR fixes CVE-2026-32282 by upgrading Go from 1.25.8 to 1.25.9.

CVE Details

  • CVE ID: CVE-2026-32282
  • Package: Go stdlib (os.Root.Chmod, internal/syscall/unix.Fchmodat)
  • Severity: Moderate
  • Impact: TOCTOU race in os.Root.Chmod allows symlink traversal and unauthorized permission changes outside a restricted directory on Linux. The Linux fchmodat syscall silently ignores AT_SYMLINK_NOFOLLOW.
  • Vulnerable versions: Go < 1.25.9
  • Fixed version: Go 1.25.9
  • Advisory: os: Root.Chmod can follow symlinks out of the root on Linux golang/go#78293
  • Jira Issue: SRVKP-11615

Additional CVEs Fixed by Go 1.25.9

This Go version bump also resolves:

  • CVE-2026-32283: crypto/tls DoS via multiple TLS 1.3 key update messages
  • CVE-2026-32281: crypto/x509 DoS via inefficient certificate chain validation

Changes

  • go.mod: Updated go 1.25.8go 1.25.9
  • Ran go mod tidy && go mod verify && go mod vendor

Test Results

Status: ✅ All tests passed

Test command: go test ./...
Result: PASSED
Duration: ~3 minutes

All packages passed, including:

  • github.com/tektoncd/cli/pkg/pods (8s)
  • github.com/tektoncd/cli/pkg/pipelinerun (4s)
  • github.com/tektoncd/cli/pkg/trustedresources (1s)

Breaking Changes

None. This is a Go patch-level update within the same minor line (1.25.x).

Testing Checklist

  • Pre-PR go test ./... passed
  • go mod tidy passed
  • go mod verify passed
  • go mod vendor passed
  • Verify CVE is resolved with security scan
  • Test affected functionality manually

Risk Assessment

Category Risk Notes
Breaking changes Low Patch-level Go update
Dependency compatibility Low Same minor line
Runtime behavior Low Security fix only

Security fix: update Go from 1.25.8 to 1.25.9 to address CVE-2026-32282 (os.Root.Chmod symlink traversal), CVE-2026-32283 (crypto/tls DoS), and CVE-2026-32281 (crypto/x509 DoS)

🤖 Generated by CVE Fixer Workflow

@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label May 19, 2026
@tekton-robot tekton-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 19, 2026
@divyansh42 divyansh42 force-pushed the fix/SRVKP-11615-cve-2026-32282-go-stdlib-release-v0.37.6-attempt-2 branch from a9cd69a to 535c2d7 Compare May 20, 2026 06:00
@tekton-robot tekton-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels May 20, 2026
@divyansh42 divyansh42 force-pushed the fix/SRVKP-11615-cve-2026-32282-go-stdlib-release-v0.37.6-attempt-2 branch from 535c2d7 to 8ffa2c8 Compare May 20, 2026 06:53
@tekton-robot tekton-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 20, 2026
@divyansh42
Copy link
Copy Markdown
Member Author

/hold

@tekton-robot tekton-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 20, 2026
…1.25.8 to 1.25.9

Signed-off-by: divyansh42 <diagrawa@redhat.com>
@divyansh42 divyansh42 force-pushed the fix/SRVKP-11615-cve-2026-32282-go-stdlib-release-v0.37.6-attempt-2 branch from 8ffa2c8 to a756a3b Compare May 20, 2026 08:16
@tekton-robot tekton-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels May 20, 2026
@divyansh42
Copy link
Copy Markdown
Member Author

/hold cancel

@tekton-robot tekton-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 20, 2026
@divyansh42
Copy link
Copy Markdown
Member Author

/retest

1 similar comment
@divyansh42
Copy link
Copy Markdown
Member Author

/retest

@pratap0007
Copy link
Copy Markdown
Contributor

/approve
/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 20, 2026
@tekton-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pratap0007

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 20, 2026
@tekton-robot tekton-robot merged commit 65a13f0 into release-v0.37.6 May 20, 2026
19 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants