Skip to content
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

Add nil check when determining whether to log best-effort program kind error #816

Merged
merged 2 commits into from
Mar 9, 2025

Conversation

egibs
Copy link
Member

@egibs egibs commented Mar 9, 2025

@jonjohnsonjr pointed out that we'll panic if c.Renderer is nil and we're unable to determine the program kind.

This PR matches similar nil checks we added previously on L442, L593, and L607.

@egibs egibs requested a review from jonjohnsonjr March 9, 2025 12:14
@egibs egibs enabled auto-merge (squash) March 9, 2025 12:15
@egibs egibs merged commit 1cd56a5 into chainguard-dev:main Mar 9, 2025
9 checks passed
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