Skip to content

Security: Fix CVE-2026-40161, CVE-2026-40938 - tektoncd/pipeline v1.3.3 → v1.3.4#3382

Merged
tekton-robot merged 2 commits intorelease-v0.77.xfrom
fix/cve-2026-40161-cve-2026-40938-tektoncd-pipeline-release-v0.77.x-attempt-1
Apr 30, 2026
Merged

Security: Fix CVE-2026-40161, CVE-2026-40938 - tektoncd/pipeline v1.3.3 → v1.3.4#3382
tekton-robot merged 2 commits intorelease-v0.77.xfrom
fix/cve-2026-40161-cve-2026-40938-tektoncd-pipeline-release-v0.77.x-attempt-1

Conversation

@jkhelil
Copy link
Copy Markdown
Member

@jkhelil jkhelil commented Apr 29, 2026

Summary

This PR fixes CVE-2026-40161 and CVE-2026-40938 by upgrading github.com/tektoncd/pipeline from v1.3.3 to v1.3.4, and updating components.yaml accordingly.

CVE Details

Field Value
CVE-2026-40161 GHSA-wjxp-xrpv-xpff
Severity HIGH
Impact Git resolver API mode leaks system-configured API token to user-controlled serverURL. A user who can create TaskRuns can exfiltrate the system Git API token by pointing the resolver at an attacker-controlled server.
Affected versions < v1.3.4
Fixed version v1.3.4
Field Value
CVE-2026-40938 GHSA-94jr-7pqp-xhcq
Severity HIGH
Impact Git resolver unsanitized revision parameter enables argument injection into the git CLI, potentially leading to remote code execution on the resolver pod.
Affected versions < v1.3.4
Fixed version v1.3.4

Changes

go.mod:       github.com/tektoncd/pipeline v1.3.3 → v1.3.4
components.yaml: pipeline.version v1.3.3 → v1.3.4
vendor/modules.txt: updated
vendor/github.com/tektoncd/pipeline/...: updated to v1.3.4

Test Results

Status: ✅ All tests passed

Test command: GOTOOLCHAIN=go1.25.0 go test ./...
Exit code: 0
Go version: go1.25.0 (matching repo go.mod)

All unit tests passed with the updated dependency. Full test suite runs in CI after PR creation.

Breaking Changes

None. This is a patch-level upgrade within the v1.3.x series. The security fixes are in the git resolver logic and do not change public APIs or behavior for non-git-resolver use cases.

Verification Steps

  • Confirm github.com/tektoncd/pipeline is at v1.3.4 in go.mod
  • Confirm components.yaml pipeline.version is v1.3.4
  • Verify CVE-2026-40161 is resolved: git resolver no longer forwards API tokens to user-controlled serverURL
  • Verify CVE-2026-40938 is resolved: revision parameter is sanitized before passing to git CLI
  • Run govulncheck ./... to confirm no remaining HIGH CVEs

Risk Assessment

Category Assessment
Risk level Low
Change scope Patch upgrade within same minor version (v1.3.x)
Breaking changes None expected
Rollback Revert to v1.3.3 if issues arise

Jira References

Resolves: SRVKP-11655, SRVKP-11656, SRVKP-11663, SRVKP-11725, SRVKP-11726, SRVKP-11733


🤖 Generated by CVE Fixer Workflow

Release Notes

Security fix: upgrade github.com/tektoncd/pipeline from v1.3.3 to v1.3.4 to address CVE-2026-40161 (HIGH) and CVE-2026-40938 (HIGH) - git resolver API token leak and argument injection enabling RCE.

… v1.3.4

- Upgrade github.com/tektoncd/pipeline from v1.3.3 to v1.3.4
- Update components.yaml pipeline.version from v1.3.3 to v1.3.4
- Run go mod tidy and go mod vendor to sync dependencies

CVE-2026-40161 (HIGH / GHSA-wjxp-xrpv-xpff):
  Git resolver API mode leaks system-configured API token to
  user-controlled serverURL. Fixed in v1.3.4.

CVE-2026-40938 (HIGH / GHSA-94jr-7pqp-xhcq):
  Git resolver unsanitized revision parameter enables argument injection
  into the git CLI, potentially leading to remote code execution on the
  resolver pod. Fixed in v1.3.4.

Resolves: SRVKP-11655, SRVKP-11656, SRVKP-11663, SRVKP-11725, SRVKP-11726, SRVKP-11733

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tekton-robot tekton-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Apr 29, 2026
@tekton-robot tekton-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Apr 29, 2026
@jkhelil jkhelil force-pushed the fix/cve-2026-40161-cve-2026-40938-tektoncd-pipeline-release-v0.77.x-attempt-1 branch from 82ed9f7 to ba558fd Compare April 29, 2026 10:35
@tekton-robot tekton-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 29, 2026
@jkhelil
Copy link
Copy Markdown
Member Author

jkhelil commented Apr 29, 2026

/approve

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 29, 2026
@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Apr 30, 2026

/approve

@tekton-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jkhelil, mbpavan

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

@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Apr 30, 2026

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 30, 2026
@tekton-robot tekton-robot merged commit 8d9ef0d into release-v0.77.x Apr 30, 2026
12 checks passed
@jkhelil jkhelil deleted the fix/cve-2026-40161-cve-2026-40938-tektoncd-pipeline-release-v0.77.x-attempt-1 branch April 30, 2026 05:16
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants