Skip to content

Fix crash verifying timestamps when no timestamp was verified#4881

Merged
Hayden-IO merged 1 commit into
mainfrom
fix-crash
May 21, 2026
Merged

Fix crash verifying timestamps when no timestamp was verified#4881
Hayden-IO merged 1 commit into
mainfrom
fix-crash

Conversation

@Hayden-IO

Copy link
Copy Markdown
Contributor

If no TSA chain in the trust root can verify a signed timestamp, then a crash would occur when Cosign tries to read one of the verified timestamps. We now throw an error if no timestamp was verified.

Summary

Release Note

Documentation

@Hayden-IO
Hayden-IO requested a review from a team as a code owner May 20, 2026 15:57
If no TSA chain in the trust root can verify a signed timestamp, then a
crash would occur when Cosign tries to read one of the verified
timestamps. We now throw an error if no timestamp was verified.

Signed-off-by: Hayden <8418760+Hayden-IO@users.noreply.github.com>
@Hayden-IO
Hayden-IO requested a review from steiza May 20, 2026 15:59
@Hayden-IO
Hayden-IO enabled auto-merge (squash) May 20, 2026 15:59

@steiza steiza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I cannot wait to delete this function. Why aren't you using the new bundle format? /s

Seriously though, thanks for fixing this.

@Hayden-IO
Hayden-IO merged commit 6ea2f18 into main May 21, 2026
29 checks passed
@Hayden-IO
Hayden-IO deleted the fix-crash branch May 21, 2026 13:30
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.

2 participants