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

fix(Source-to-Sink): handle empty token list in PPI find method #55

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

scriptprivate
Copy link
Contributor

  • prevent runtime error when no tokens are found by using an empty array reference as a fallback

- prevent runtime error when no tokens are found by using an empty array reference as a fallback
@htrgouvea htrgouvea merged commit 65b540f into htrgouvea:develop Dec 18, 2024
2 of 4 checks passed
htrgouvea added a commit that referenced this pull request Dec 24, 2024
* return feature to remove blank lines and comments

* add samples to perform tests

* pushing some tools to help during debug tasks

* skip false positives - draft function

* improve taint analysis function

* drafting new rules

* update sarif output with new variables

* apply design pattern practices to a better code compreension

* remove Data::Dumper

* fixed sarif

* remove unecessary variables

* remove old file

* remove samples

* create some unit tests

* update rules on linter

* resolv linter warnings

* resolv linter warnings

* new module

* remove tools/

* tdy

* update perltidyrc

* new line

* deleted tests/Sarif.t

* remove blank lines

* checking if the name of file that does exists

* fix(Source-to-Sink): handle empty token list in PPI find method (#55)

* fix(Source-to-Sink): handle empty token list in PPI find method

- prevent runtime error when no tokens are found by using an empty array reference as a fallback

* update security-gate.yml

---------

Co-authored-by: Heitor <[email protected]>
Co-authored-by: priv <[email protected]>
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