Skip to content

SQL test expected issue file fix - #472

Open
SampathKumarAmex wants to merge 1 commit into
SasanLabs:masterfrom
SampathKumarAmex:sql-inject-fix
Open

SQL test expected issue file fix#472
SampathKumarAmex wants to merge 1 commit into
SasanLabs:masterfrom
SampathKumarAmex:sql-inject-fix

Conversation

@SampathKumarAmex

Copy link
Copy Markdown
Contributor

Fixes one of the issues highlighted : #448

The line of code is the beginning of a try-catch block:
SQL Injection : src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java : 218

CWE-89,SQL Injection,src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java,115,1
CWE-89,SQL Injection,src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java,165,1
CWE-89,SQL Injection,src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java,218,1
CWE-89,SQL Injection,src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java,211,1

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 think it should be 209.

@github-actions

Copy link
Copy Markdown
Contributor

This PR has had no activity for 30 days, so it's been marked stale. If you're still working on it, push a commit or leave a comment and this label will clear. Otherwise it'll close automatically in 60 days — no hard feelings, and you (or anyone else) are welcome to reopen or resubmit against current main whenever you're ready.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants