Skip to content

Commit 30249e4

Browse files
committed
Java: Add some spurious source and sink examples.
1 parent 854674a commit 30249e4

File tree

4 files changed

+30
-0
lines changed

4 files changed

+30
-0
lines changed

java/ql/test/utils/modelgenerator/dataflow/CaptureSinkModels.ext.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ extensions:
55
extensible: sinkModel
66
data:
77
- [ "p", "Sinks", False, "sink", "(Object)", "", "Argument[0]", "test-sink", "manual" ]
8+
- [ "p", "Sinks", False, "manualSinkAlreadyDefined", "(Object)", "", "Argument[0]", "test-sink", "manual" ]
89

910
- addsTo:
1011
pack: codeql/java-all

java/ql/test/utils/modelgenerator/dataflow/CaptureSourceModels.ext.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,10 @@ extensions:
55
extensible: sourceModel
66
data:
77
- [ "p", "Sources", False, "source", "()", "", "ReturnValue", "test-source", "manual" ]
8+
- [ "p", "Sources", False, "manualSourceAlreadyDefined", "()", "", "ReturnValue", "test-source", "manual" ]
9+
10+
- addsTo:
11+
pack: codeql/java-all
12+
extensible: neutralModel
13+
data:
14+
- ["p", "Sources", "manualNeutralSource", "()", "source", "manual"]

java/ql/test/utils/modelgenerator/dataflow/p/Sinks.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,4 +71,11 @@ public void wrapSinkSimpleType(String s) {
7171
Boolean b = s == "hello";
7272
sink(b);
7373
}
74+
75+
// Not a new sink as this callable already has a manual sink.
76+
// SPURIOUS-sink=p;Sinks;true;manualSinkAlreadyDefined;(Object);;Argument[0];test-sink;df-generated
77+
// neutral=p;Sinks;manualSinkAlreadyDefined;(Object);summary;df-generated
78+
public void manualSinkAlreadyDefined(Object o) {
79+
sink(o);
80+
}
7481
}

java/ql/test/utils/modelgenerator/dataflow/p/Sources.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,4 +66,19 @@ public String read() {
6666
return value.toString();
6767
}
6868
}
69+
70+
// Not a new source as this callable has been manually modelled
71+
// as source neutral.
72+
// SPURIOUS-source=p;Sources;true;manualNeutralSource;();;ReturnValue;test-source;df-generated
73+
// neutral=p;Sources;manualNeutralSource;();summary;df-generated
74+
public String manualNeutralSource() {
75+
return source();
76+
}
77+
78+
// Not a new source as this callable already has a manual source.
79+
// SPURIOUS-source=p;Sources;true;manualSourceAlreadyDefined;();;ReturnValue;test-source;df-generated
80+
// neutral=p;Sources;manualSourceAlreadyDefined;();summary;df-generated
81+
public String manualSourceAlreadyDefined() {
82+
return source();
83+
}
6984
}

0 commit comments

Comments
 (0)