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 exclusions to data flow consistency checks

Browse Source
This commit is contained in:
Owen Mansel-Chan
2025-11-26 10:14:03 +00:00
parent a2e68487ea
commit eca9ec59c4
198 changed files with 15 additions and 1471519 deletions
Download Patch File Download Diff File Expand all files Collapse all files

49
go/ql/test/query-tests/InconsistentCode/ConstantLengthComparison/CONSISTENCY/DataFlowConsistency.expected
View File

@@ -1,49 +0,0 @@
uniqueNodeLocation
| file://:0:0:0:0 | [summary param] 0 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[0].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0] in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue.ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | function append | Node should have one location but has 0. |
| file://:0:0:0:0 | function cap | Node should have one location but has 0. |
| file://:0:0:0:0 | function clear | Node should have one location but has 0. |
| file://:0:0:0:0 | function close | Node should have one location but has 0. |
| file://:0:0:0:0 | function complex | Node should have one location but has 0. |
| file://:0:0:0:0 | function copy | Node should have one location but has 0. |
| file://:0:0:0:0 | function delete | Node should have one location but has 0. |
| file://:0:0:0:0 | function imag | Node should have one location but has 0. |
| file://:0:0:0:0 | function len | Node should have one location but has 0. |
| file://:0:0:0:0 | function make | Node should have one location but has 0. |
| file://:0:0:0:0 | function max | Node should have one location but has 0. |
| file://:0:0:0:0 | function min | Node should have one location but has 0. |
| file://:0:0:0:0 | function new | Node should have one location but has 0. |
| file://:0:0:0:0 | function panic | Node should have one location but has 0. |
| file://:0:0:0:0 | function print | Node should have one location but has 0. |
| file://:0:0:0:0 | function println | Node should have one location but has 0. |
| file://:0:0:0:0 | function real | Node should have one location but has 0. |
| file://:0:0:0:0 | function recover | Node should have one location but has 0. |
missingLocation
| Nodes without location: 37 |
localFlowIsLocal
| file://:0:0:0:0 | function len | ConstantLengthComparison.go:4:18:4:20 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | ConstantLengthComparison.go:5:6:5:8 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | ConstantLengthComparisonGood.go:4:18:4:20 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | ConstantLengthComparisonGood.go:5:6:5:8 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | tst.go:4:5:4:7 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | tst.go:5:10:5:12 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | tst.go:8:18:8:20 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | tst.go:9:6:9:8 | len | Local flow step does not preserve enclosing callable. |

54
go/ql/test/query-tests/InconsistentCode/InconsistentLoopOrientation/CONSISTENCY/DataFlowConsistency.expected
View File

@@ -1,57 +1,3 @@
uniqueNodeLocation
| file://:0:0:0:0 | [summary param] 0 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 3 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 3 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[0].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0] in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue.ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | function append | Node should have one location but has 0. |
| file://:0:0:0:0 | function cap | Node should have one location but has 0. |
| file://:0:0:0:0 | function clear | Node should have one location but has 0. |
| file://:0:0:0:0 | function close | Node should have one location but has 0. |
| file://:0:0:0:0 | function complex | Node should have one location but has 0. |
| file://:0:0:0:0 | function copy | Node should have one location but has 0. |
| file://:0:0:0:0 | function delete | Node should have one location but has 0. |
| file://:0:0:0:0 | function imag | Node should have one location but has 0. |
| file://:0:0:0:0 | function len | Node should have one location but has 0. |
| file://:0:0:0:0 | function make | Node should have one location but has 0. |
| file://:0:0:0:0 | function max | Node should have one location but has 0. |
| file://:0:0:0:0 | function min | Node should have one location but has 0. |
| file://:0:0:0:0 | function new | Node should have one location but has 0. |
| file://:0:0:0:0 | function panic | Node should have one location but has 0. |
| file://:0:0:0:0 | function print | Node should have one location but has 0. |
| file://:0:0:0:0 | function println | Node should have one location but has 0. |
| file://:0:0:0:0 | function real | Node should have one location but has 0. |
| file://:0:0:0:0 | function recover | Node should have one location but has 0. |
missingLocation
| Nodes without location: 39 |
localFlowIsLocal
| file://:0:0:0:0 | function len | InconsistentLoopOrientation.go:10:26:10:28 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | InconsistentLoopOrientationGood.go:10:26:10:28 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:9:22:9:24 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:14:17:14:19 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:25:26:25:28 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:31:31:31:33 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:37:16:37:18 | len | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function len | main.go:37:38:37:40 | len | Local flow step does not preserve enclosing callable. |
argHasPostUpdate
| main.go:9:26:9:26 | s | ArgumentNode is missing PostUpdateNode. |
| main.go:14:21:14:21 | s | ArgumentNode is missing PostUpdateNode. |
identityLocalStep
| InconsistentLoopOrientation.go:6:3:6:3 | a | Node steps to itself |
| InconsistentLoopOrientationGood.go:6:3:6:3 | a | Node steps to itself |

4970
go/ql/test/query-tests/InconsistentCode/LengthComparisonOffByOne/CONSISTENCY/DataFlowConsistency.expected
View File

File diff suppressed because it is too large Load Diff

6494
go/ql/test/query-tests/InconsistentCode/MissingErrorCheck/CONSISTENCY/DataFlowConsistency.expected
View File

File diff suppressed because it is too large Load Diff

6385
go/ql/test/query-tests/InconsistentCode/MistypedExponentiation/CONSISTENCY/DataFlowConsistency.expected
View File

File diff suppressed because it is too large Load Diff

6495
go/ql/test/query-tests/InconsistentCode/UnhandledCloseWritableHandle/CONSISTENCY/DataFlowConsistency.expected
View File

File diff suppressed because it is too large Load Diff

42
go/ql/test/query-tests/InconsistentCode/WhitespaceContradictsPrecedence/CONSISTENCY/DataFlowConsistency.expected
View File

@@ -1,42 +0,0 @@
uniqueNodeLocation
| file://:0:0:0:0 | [summary param] 0 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 3 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 3 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[0].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0] in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue.ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | function append | Node should have one location but has 0. |
| file://:0:0:0:0 | function cap | Node should have one location but has 0. |
| file://:0:0:0:0 | function clear | Node should have one location but has 0. |
| file://:0:0:0:0 | function close | Node should have one location but has 0. |
| file://:0:0:0:0 | function complex | Node should have one location but has 0. |
| file://:0:0:0:0 | function copy | Node should have one location but has 0. |
| file://:0:0:0:0 | function delete | Node should have one location but has 0. |
| file://:0:0:0:0 | function imag | Node should have one location but has 0. |
| file://:0:0:0:0 | function len | Node should have one location but has 0. |
| file://:0:0:0:0 | function make | Node should have one location but has 0. |
| file://:0:0:0:0 | function max | Node should have one location but has 0. |
| file://:0:0:0:0 | function min | Node should have one location but has 0. |
| file://:0:0:0:0 | function new | Node should have one location but has 0. |
| file://:0:0:0:0 | function panic | Node should have one location but has 0. |
| file://:0:0:0:0 | function print | Node should have one location but has 0. |
| file://:0:0:0:0 | function println | Node should have one location but has 0. |
| file://:0:0:0:0 | function real | Node should have one location but has 0. |
| file://:0:0:0:0 | function recover | Node should have one location but has 0. |
missingLocation
| Nodes without location: 39 |

68
go/ql/test/query-tests/InconsistentCode/WrappedErrorAlwaysNil/CONSISTENCY/DataFlowConsistency.expected
View File

@@ -1,68 +0,0 @@
uniqueNodeLocation
| file://:0:0:0:0 | [summary param] 0 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 0 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 1 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary param] 2 in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[0].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] read: Argument[1].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0] in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: Argument[0].ArrayElement in copy | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in append | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in max | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue in min | Node should have one location but has 0. |
| file://:0:0:0:0 | [summary] to write: ReturnValue.ArrayElement in append | Node should have one location but has 0. |
| file://:0:0:0:0 | function Error | Node should have one location but has 0. |
| file://:0:0:0:0 | function Errorf | Node should have one location but has 0. |
| file://:0:0:0:0 | function Wrap | Node should have one location but has 0. |
| file://:0:0:0:0 | function append | Node should have one location but has 0. |
| file://:0:0:0:0 | function cap | Node should have one location but has 0. |
| file://:0:0:0:0 | function clear | Node should have one location but has 0. |
| file://:0:0:0:0 | function close | Node should have one location but has 0. |
| file://:0:0:0:0 | function complex | Node should have one location but has 0. |
| file://:0:0:0:0 | function copy | Node should have one location but has 0. |
| file://:0:0:0:0 | function delete | Node should have one location but has 0. |
| file://:0:0:0:0 | function imag | Node should have one location but has 0. |
| file://:0:0:0:0 | function len | Node should have one location but has 0. |
| file://:0:0:0:0 | function make | Node should have one location but has 0. |
| file://:0:0:0:0 | function max | Node should have one location but has 0. |
| file://:0:0:0:0 | function min | Node should have one location but has 0. |
| file://:0:0:0:0 | function new | Node should have one location but has 0. |
| file://:0:0:0:0 | function panic | Node should have one location but has 0. |
| file://:0:0:0:0 | function print | Node should have one location but has 0. |
| file://:0:0:0:0 | function println | Node should have one location but has 0. |
| file://:0:0:0:0 | function real | Node should have one location but has 0. |
| file://:0:0:0:0 | function recover | Node should have one location but has 0. |
missingLocation
| Nodes without location: 40 |
localFlowIsLocal
| WrappedErrorAlwaysNil.go:9:6:9:7 | function f1 | WrappedErrorAlwaysNil.go:24:9:24:10 | f1 | Local flow step does not preserve enclosing callable. |
| WrappedErrorAlwaysNil.go:16:6:16:7 | function f2 | WrappedErrorAlwaysNil.go:29:15:29:16 | f2 | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Errorf | WrappedErrorAlwaysNil.go:11:10:11:22 | selection of Errorf | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Errorf | WrappedErrorAlwaysNil.go:18:17:18:29 | selection of Errorf | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:27:10:27:20 | selection of Wrap | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:31:10:31:20 | selection of Wrap | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:38:2:38:12 | selection of Wrap | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:41:2:41:12 | selection of Wrap | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:45:2:45:12 | selection of Wrap | Local flow step does not preserve enclosing callable. |
| file://:0:0:0:0 | function Wrap | WrappedErrorAlwaysNil.go:49:2:49:12 | selection of Wrap | Local flow step does not preserve enclosing callable. |
argHasPostUpdate
| WrappedErrorAlwaysNil.go:11:10:11:37 | []type{args} | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:11:24:11:36 | "error in f1" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:18:17:18:44 | []type{args} | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:18:31:18:43 | "error in f2" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:24:12:24:16 | input | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:27:27:27:28 | "" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:29:18:29:22 | input | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:31:27:31:28 | "" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:38:19:38:20 | "" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:41:14:41:16 | nil | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:41:19:41:20 | "" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:45:19:45:20 | "" | ArgumentNode is missing PostUpdateNode. |
| WrappedErrorAlwaysNil.go:49:24:49:25 | "" | ArgumentNode is missing PostUpdateNode. |