Skip to content

Commit d544cd5

Browse files
authored
Merge pull request #3671 from semgrep/kb/sqli-fmt-string-mvar-focus
fix(java): formatted-sql-string Java rule had incorrect mvar-focus
2 parents e959640 + 0cae354 commit d544cd5

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

java/lang/security/audit/formatted-sql-string.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,3 +143,13 @@ public List<Student> addWhere(String name, CriteriaQuery Query)
143143
return students;
144144
}
145145
}
146+
147+
public class SqlExampleFocusMetavar {
148+
public void get(HttpServletRequest req) {
149+
Connection c = DB.getConnection();
150+
String p = req.getParam("param");
151+
PreparedStatement statement = c.prepareStatment("SELECT * FROM " + p);
152+
// ruleid: formatted-sql-string
153+
ResultSet rs = statement.executeQuery();
154+
}
155+
}

java/lang/security/audit/formatted-sql-string.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ rules:
4646
...
4747
}
4848
- pattern: (String $INPUT)
49-
- focus-metavariable: $INPUT
49+
- focus-metavariable: $INPUT
5050
label: INPUT
5151
- patterns:
5252
- pattern-either:

0 commit comments

Comments
 (0)