Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

fix: catches nil package to resolve panic in TaintAnalyzer #84

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: catches nil package to resolve panic in TaintAnalyzer #84

wants to merge 1 commit into from

Conversation

smoyer64
Copy link

Resolves #83

@smoyer64 smoyer64 mentioned this pull request Jan 2, 2023
Closed
glinton
glinton approved these changes Jan 13, 2023
View reviewed changes
Copy link

@glinton glinton left a comment

Choose a reason for hiding this comment

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

Code fix looks fine to me. If I was maintaining this repo, I'd love to see a test added that verifies this change to prevent a regression.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@glinton glinton glinton approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GoKart panics in the TaintAnalyzer
2 participants
@smoyer64 @glinton