Skip to content

Commit 8113d1f

Browse files
committed
adding loop bound injection
1 parent f2a9a69 commit 8113d1f

File tree

3 files changed

+22
-7
lines changed

3 files changed

+22
-7
lines changed

vulnerabilities/exec.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,4 +36,6 @@ function runMe(cmd,res){
3636
cmdRunning.on('close', (code) => {
3737
res.send(`child process exited with code ${code}`);
3838
});
39-
}
39+
}
40+
41+
module.exports = router

vulnerabilities/lib/db.js

Lines changed: 0 additions & 6 deletions
This file was deleted.

vulnerabilities/loop.js

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
const express = require('express');
2+
const router = express.Router()
3+
4+
5+
router.post("/list-users", (req, res) => {
6+
var obj = req.body.users;
7+
var someArr = [];
8+
9+
// Potential DoS if obj.length is large.
10+
for (var i = 0; i < obj.length; i++) {
11+
someArr.push(obj[i]);
12+
}
13+
14+
//doing something with the code
15+
res.send(someArr.join(','));
16+
});
17+
18+
19+
module.exports = router

0 commit comments

Comments
 (0)