Skip to content

Commit 75ed0d0

Browse files
committed
JS: Remove duplicate '$ Alert' in libxml test
1 parent 6a47678 commit 75ed0d0

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

javascript/ql/test/query-tests/Security/CWE-611/libxml.noent.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,14 @@ const express = require('express');
22
const libxmljs = require('libxmljs');
33

44
express().get('/some/path', function (req) {
5-
libxmljs.parseXml(req.param("some-xml"), { noent: true }); // $ Alert // $ Alert - unguarded entity expansion
5+
libxmljs.parseXml(req.param("some-xml"), { noent: true }); // $ Alert - unguarded entity expansion
66
});
77

88
express().post('/some/path', function (req, res) {
9-
libxmljs.parseXml(req.param("some-xml"), { noent: true }); // $ Alert // $ Alert - unguarded entity expansion
9+
libxmljs.parseXml(req.param("some-xml"), { noent: true }); // $ Alert - unguarded entity expansion
1010

11-
libxmljs.parseXmlString(req.param("some-xml"), { noent: true }) // $ Alert // $ Alert - unguarded entity expansion
12-
libxmljs.parseXmlString(req.files.products.data.toString('utf8'), { noent: true })// $ Alert // $ Alert - unguarded entity expansion
11+
libxmljs.parseXmlString(req.param("some-xml"), { noent: true }) // $ Alert - unguarded entity expansion
12+
libxmljs.parseXmlString(req.files.products.data.toString('utf8'), { noent: true })// $ Alert - unguarded entity expansion
1313

1414
// OK - no entity expansion
1515
libxmljs.parseXmlString(req.files.products.data.toString('utf8'), { noent: false })

0 commit comments

Comments
 (0)