Skip to content

Commit 32d6ac8

Browse files
committed
Add test case to ensure exec calls without middleware injection into Express are not flagged.
1 parent 45c8ec9 commit 32d6ac8

File tree

1 file changed

+9
-0
lines changed
  • javascript/ql/test/query-tests/Security/CWE-089/untyped

1 file changed

+9
-0
lines changed

Diff for: javascript/ql/test/query-tests/Security/CWE-089/untyped/hana.js

+9
Original file line numberDiff line numberDiff line change
@@ -84,3 +84,12 @@ app2.post('/documents/find', (req, res) => {
8484
client.execute('select A, B from TEST.NUMBERS order by A' + maliciousInput, function(err, rs) {}); // $ Alert
8585
});
8686
});
87+
88+
var app3 = express();
89+
90+
app3.get('/execute-query', function (req, res) {
91+
var client = req.db;
92+
let maliciousInput = req.body.data;
93+
client.exec('SELECT * FROM DUMMY' + maliciousInput, function (err, rs) {});
94+
req.db.exec('SELECT * FROM DUMMY' + maliciousInput, function (err, rs) {});
95+
});

0 commit comments

Comments
 (0)