Skip to content

Commit 21bcaf6

Browse files
committed
C#/Java: After remaining code after rebase.
1 parent 7435292 commit 21bcaf6

File tree

5 files changed

+19
-19
lines changed

5 files changed

+19
-19
lines changed

csharp/ql/src/utils/model-generator/CaptureSummaryModels.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,11 @@ private import CaptureSummaryModels
8181
* Captured Model:
8282
* ```Summaries;BasicFlow;false;AssignToArray;(System.Int32,System.Int32[]);Argument[0];Argument[1].Element;taint```
8383
*/
84-
private string captureFlow(TargetAPI api) {
84+
private string captureFlow(TargetApi api) {
8585
result = captureQualifierFlow(api) or
8686
result = captureThroughFlow(api)
8787
}
8888

89-
from TargetAPI api, string flow
89+
from TargetApi api, string flow
9090
where flow = captureFlow(api)
9191
select flow order by flow

csharp/ql/src/utils/model-generator/CaptureSummaryModels.qll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import CaptureSummaryModelsSpecific
88
/**
99
* Gets the summary model of `api`, if it follows the `fluent` programming pattern (returns `this`).
1010
*/
11-
string captureQualifierFlow(TargetAPI api) {
11+
string captureQualifierFlow(TargetApi api) {
1212
exists(ReturnNodeExt ret |
1313
api = returnNodeEnclosingCallable(ret) and
1414
isOwnInstanceAccessNode(ret)
@@ -41,7 +41,7 @@ class ThroughFlowConfig extends TaintTracking::Configuration {
4141

4242
override predicate isSource(DataFlow::Node source, DataFlow::FlowState state) {
4343
source instanceof DataFlow::ParameterNode and
44-
source.getEnclosingCallable() instanceof TargetAPI and
44+
source.getEnclosingCallable() instanceof TargetApi and
4545
state instanceof TaintRead
4646
}
4747

@@ -52,7 +52,7 @@ class ThroughFlowConfig extends TaintTracking::Configuration {
5252
(state instanceof TaintRead or state instanceof TaintStore)
5353
}
5454

55-
override predicate isAdditionalFlowStep(
55+
override predicate isAdditionalTaintStep(
5656
DataFlow::Node node1, DataFlow::FlowState state1, DataFlow::Node node2,
5757
DataFlow::FlowState state2
5858
) {
@@ -83,7 +83,7 @@ class ThroughFlowConfig extends TaintTracking::Configuration {
8383
/**
8484
* Gets the summary model(s) of `api`, if there is flow from parameters to return value or parameter.
8585
*/
86-
string captureThroughFlow(TargetAPI api) {
86+
string captureThroughFlow(TargetApi api) {
8787
exists(
8888
ThroughFlowConfig config, DataFlow::ParameterNode p, ReturnNodeExt returnNodeExt, string input,
8989
string output

csharp/ql/src/utils/model-generator/ModelGeneratorUtils.qll

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ predicate isRelevantContent(DataFlow::Content c) {
3131
* Gets the summary model for `api` with `input`, `output` and `kind`.
3232
*/
3333
bindingset[input, output, kind]
34-
string asSummaryModel(TargetAPI api, string input, string output, string kind) {
34+
string asSummaryModel(TargetApi api, string input, string output, string kind) {
3535
result =
3636
asPartialModel(api) + input + ";" //
3737
+ output + ";" //
@@ -42,30 +42,30 @@ string asSummaryModel(TargetAPI api, string input, string output, string kind) {
4242
* Gets the value summary model for `api` with `input` and `output`.
4343
*/
4444
bindingset[input, output]
45-
string asValueModel(TargetAPI api, string input, string output) {
45+
string asValueModel(TargetApi api, string input, string output) {
4646
result = asSummaryModel(api, input, output, "value")
4747
}
4848

4949
/**
5050
* Gets the taint summary model for `api` with `input` and `output`.
5151
*/
5252
bindingset[input, output]
53-
string asTaintModel(TargetAPI api, string input, string output) {
53+
string asTaintModel(TargetApi api, string input, string output) {
5454
result = asSummaryModel(api, input, output, "taint")
5555
}
5656

5757
/**
5858
* Gets the sink model for `api` with `input` and `kind`.
5959
*/
6060
bindingset[input, kind]
61-
string asSinkModel(TargetAPI api, string input, string kind) {
61+
string asSinkModel(TargetApi api, string input, string kind) {
6262
result = asPartialModel(api) + input + ";" + kind
6363
}
6464

6565
/**
6666
* Gets the source model for `api` with `output` and `kind`.
6767
*/
6868
bindingset[output, kind]
69-
string asSourceModel(TargetAPI api, string output, string kind) {
69+
string asSourceModel(TargetApi api, string output, string kind) {
7070
result = asPartialModel(api) + output + ";" + kind
7171
}

csharp/ql/src/utils/model-generator/ModelGeneratorUtilsSpecific.qll

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,15 @@ private predicate isRelevantForModels(Callable api) { not api instanceof MainMet
1212
* In the Standard library and 3rd party libraries it the Callables that can be called
1313
* from outside the library itself.
1414
*/
15-
class TargetAPI extends Callable {
16-
TargetAPI() {
15+
class TargetApi extends Callable {
16+
TargetApi() {
1717
[this.(Modifiable), this.(Accessor).getDeclaration()].isEffectivelyPublic() and
1818
this.fromSource() and
1919
isRelevantForModels(this)
2020
}
2121
}
2222

23-
private string parameterQualifiedTypeNamesToString(TargetAPI api) {
23+
private string parameterQualifiedTypeNamesToString(TargetApi api) {
2424
result =
2525
concat(Parameter p, int i |
2626
p = api.getParameter(i)
@@ -30,7 +30,7 @@ private string parameterQualifiedTypeNamesToString(TargetAPI api) {
3030
}
3131

3232
/** Holds if the summary should apply for all overrides of this. */
33-
private predicate isBaseCallableOrPrototype(TargetAPI api) {
33+
private predicate isBaseCallableOrPrototype(TargetApi api) {
3434
api.getDeclaringType() instanceof Interface
3535
or
3636
exists(Modifiable m | m = [api.(Modifiable), api.(Accessor).getDeclaration()] |
@@ -41,12 +41,12 @@ private predicate isBaseCallableOrPrototype(TargetAPI api) {
4141
}
4242

4343
/** Gets a string representing whether the summary should apply for all overrides of this. */
44-
private string getCallableOverride(TargetAPI api) {
44+
private string getCallableOverride(TargetApi api) {
4545
if isBaseCallableOrPrototype(api) then result = "true" else result = "false"
4646
}
4747

4848
/** Computes the first 6 columns for CSV rows. */
49-
string asPartialModel(TargetAPI api) {
49+
string asPartialModel(TargetApi api) {
5050
exists(string namespace, string type |
5151
api.getDeclaringType().hasQualifiedName(namespace, type) and
5252
result =

java/ql/src/utils/model-generator/CaptureSummaryModels.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,11 @@ private import CaptureSummaryModels
7373
* Captured Model:
7474
* ```p;Foo;true;addToList;;Argument[0];Argument[1];taint```
7575
*/
76-
string captureFlow(TargetAPI api) {
76+
string captureFlow(TargetApi api) {
7777
result = captureQualifierFlow(api) or
7878
result = captureThroughFlow(api)
7979
}
8080

81-
from TargetAPI api, string flow
81+
from TargetApi api, string flow
8282
where flow = captureFlow(api)
8383
select flow order by flow

0 commit comments

Comments
 (0)