hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-30 19:26:02 +02:00
Code Issues Packages Projects Releases Wiki Activity

add support for the lie polyfill

Browse Source
This commit is contained in:
Erik Krogh Kristensen
2021-06-20 19:49:20 +02:00
parent 085efe5d20
commit 95a7b16315
4 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
javascript/change-notes/2021-06-18-promises.md
View File

@@ -11,4 +11,5 @@ lgtm,codescanning
[pinkie-promise](https://npmjs.com/package/pinkie-promise),
[pinkie](https://npmjs.com/package/pinkie),
[synchronous-promise](https://npmjs.com/package/synchronous-promise),
[any-promise](https://npmjs.com/package/any-promise)
[any-promise](https://npmjs.com/package/any-promise),
[lie](https://npmjs.com/package/lie)

2
javascript/ql/src/semmle/javascript/Promises.qll
View File

@@ -91,6 +91,8 @@ private DataFlow::SourceNode getAPromiseObject() {
result = DataFlow::moduleMember("synchronous-promise", "SynchronousPromise")
or
result = DataFlow::moduleImport("any-promise")
or
result = DataFlow::moduleImport("lie")
}
/**

5
javascript/ql/test/library-tests/Promises/promises.js
View File

@@ -146,4 +146,9 @@
(function() {
var Promise = require('any-promise');
return new Promise(function(resolve, reject){})
})();
(function() {
var Promise = require('lie');
var promise = Promise.resolve(source);
})();

3
javascript/ql/test/library-tests/Promises/tests.expected
View File

@@ -44,6 +44,7 @@ test_ResolvedPromiseDefinition
| promises.js:119:3:119:25 | Promise ... source) | promises.js:119:19:119:24 | source |
| promises.js:125:20:125:39 | when.resolve(source) | promises.js:125:33:125:38 | source |
| promises.js:143:17:143:50 | Synchro ... source) | promises.js:143:44:143:49 | source |
| promises.js:153:17:153:39 | Promise ... source) | promises.js:153:33:153:38 | source |
test_PromiseDefinition_getARejectHandler
| flow.js:26:2:26:49 | new Pro ... ource)) | flow.js:26:69:26:80 | y => sink(y) |
| flow.js:32:2:32:49 | new Pro ... ource)) | flow.js:32:57:32:68 | x => sink(x) |
@@ -449,3 +450,5 @@ typetrack
| promises.js:135:3:137:4 | new Pro ... );\\n }) | promises.js:136:13:136:16 | data | store $PromiseResolveField$ |
| promises.js:143:17:143:50 | Synchro ... source) | promises.js:143:44:143:49 | source | copy $PromiseResolveField$ |
| promises.js:143:17:143:50 | Synchro ... source) | promises.js:143:44:143:49 | source | store $PromiseResolveField$ |
| promises.js:153:17:153:39 | Promise ... source) | promises.js:153:33:153:38 | source | copy $PromiseResolveField$ |
| promises.js:153:17:153:39 | Promise ... source) | promises.js:153:33:153:38 | source | store $PromiseResolveField$ |