hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-21 19:26:31 +01:00
Code Issues Packages Projects Releases Wiki Activity

JS: Ignore strict-mode-call-stack-introspection for expr stmts

Browse Source
This commit is contained in:
Asger Feldthaus
2020-01-13 15:50:30 +00:00
parent ad0ad3a3e4
commit 73e60a7400
4 changed files with 16 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
change-notes/1.24/analysis-javascript.md
View File

@@ -29,6 +29,7 @@
| Incomplete string escaping or encoding (`js/incomplete-sanitization`) | Fewer false positive results | This query now recognizes additional cases where a single replacement is likely to be intentional. | | Incomplete string escaping or encoding (`js/incomplete-sanitization`) | Fewer false positive results | This query now recognizes additional cases where a single replacement is likely to be intentional. |
| Unbound event handler receiver (`js/unbound-event-handler-receiver`) | Fewer false positive results | This query now recognizes additional ways event handler receivers can be bound. | | Unbound event handler receiver (`js/unbound-event-handler-receiver`) | Fewer false positive results | This query now recognizes additional ways event handler receivers can be bound. |
| Expression has no effect (`js/useless-expression`) | Fewer false positive results | The query now recognizes block-level flow type annotations. | | Expression has no effect (`js/useless-expression`) | Fewer false positive results | The query now recognizes block-level flow type annotations. |
| Use of call stack introspection in strict mode (`js/strict-mode-call-stack-introspection`) | Fewer false positive results | The query no longer flags expression statements. |
## Changes to libraries ## Changes to libraries

3
javascript/ql/src/LanguageFeatures/StrictModeCallStackIntrospection.ql
View File

@@ -32,5 +32,6 @@ where
acc.accesses(baseNode.asExpr(), prop) and acc.accesses(baseNode.asExpr(), prop) and
acc.getContainer().isStrict() and acc.getContainer().isStrict() and
illegalPropAccess(baseNode.getAValue(), base, prop) and illegalPropAccess(baseNode.getAValue(), base, prop) and
forex(AbstractValue av | av = baseNode.getAValue() | illegalPropAccess(av, _, prop)) forex(AbstractValue av | av = baseNode.getAValue() | illegalPropAccess(av, _, prop)) and
not acc = any(ExprStmt stmt).getExpr() // reported by js/useless-expression
select acc, "Strict mode code cannot use " + base + "." + prop + "." select acc, "Strict mode code cannot use " + base + "." + prop + "."

8
javascript/ql/test/query-tests/LanguageFeatures/StrictModeCallStackIntrospection/StrictModeCallStackIntrospection.expected
View File

@@ -1,7 +1,7 @@
| tst.js:5:30:5:45 | arguments.callee | Strict mode code cannot use arguments.callee. | | tst.js:5:30:5:45 | arguments.callee | Strict mode code cannot use arguments.callee. |
| tst.js:7:21:7:36 | arguments.callee | Strict mode code cannot use arguments.callee. | | tst.js:7:21:7:36 | arguments.callee | Strict mode code cannot use arguments.callee. |
| tst.js:9:20:9:27 | f.caller | Strict mode code cannot use Function.prototype.caller. | | tst.js:9:20:9:27 | f.caller | Strict mode code cannot use Function.prototype.caller. |
| tst.js:11:8:11:18 | f.arguments | Strict mode code cannot use Function.prototype.arguments. | | tst.js:11:17:11:27 | f.arguments | Strict mode code cannot use Function.prototype.arguments. |
| tst.js:18:3:18:18 | arguments.callee | Strict mode code cannot use arguments.callee. | | tst.js:18:10:18:25 | arguments.callee | Strict mode code cannot use arguments.callee. |
| tst.js:31:5:31:14 | foo.caller | Strict mode code cannot use Function.prototype.caller. | | tst.js:31:12:31:21 | foo.caller | Strict mode code cannot use Function.prototype.caller. |
| tst.js:31:5:31:14 | foo.caller | Strict mode code cannot use arguments.caller. | | tst.js:31:12:31:21 | foo.caller | Strict mode code cannot use arguments.caller. |

13
javascript/ql/test/query-tests/LanguageFeatures/StrictModeCallStackIntrospection/tst.js
View File

@@ -8,14 +8,14 @@ var o = {
// BAD // BAD
console.log(f.caller); console.log(f.caller);
// BAD // BAD
f.arguments; this.y = f.arguments;
this.x = x; this.x = x;
} }
}; };
var D = class extends function() { var D = class extends function() {
// BAD // BAD
arguments.callee; return arguments.callee;
} {}; } {};
function g() { function g() {
@@ -28,6 +28,11 @@ function g() {
function h() { function h() {
var foo = Math.random() > 0.5 ? h : arguments; var foo = Math.random() > 0.5 ? h : arguments;
// BAD // BAD
foo.caller; return foo.caller;
} }
})(); })();
(function() {
'use strict';
arguments.caller; // OK - avoid duplicate alert from useless-expression
})();