Skip to content

Commit 8fee9cb

Browse files
committed
Fix CodeQL warnings
1 parent c2035e8 commit 8fee9cb

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

javascript/ql/experimental/adaptivethreatmodeling/lib/experimental/adaptivethreatmodeling/ATMConfig.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ abstract class AtmConfig extends string {
7070
* an effective sink, i.e. one considered as a possible sink of flow in the boosted query.
7171
*/
7272
final predicate isEffectiveSink(JS::DataFlow::Node candidateSink) {
73-
not exists(getAReasonSinkExcluded(candidateSink))
73+
not exists(this.getAReasonSinkExcluded(candidateSink))
7474
}
7575

7676
final EndpointCharacteristics::EndpointCharacteristic getAReasonSinkExcluded(
@@ -85,7 +85,7 @@ abstract class AtmConfig extends string {
8585
or
8686
exists(EndpointCharacteristics::EndpointFilterCharacteristic specificFilter |
8787
specificFilter.getEndpoints(candidateSink) and
88-
specificFilter.getImplications(getASinkEndpointType(), false, _) and
88+
specificFilter.getImplications(this.getASinkEndpointType(), false, _) and
8989
result = specificFilter
9090
)
9191
}

javascript/ql/experimental/adaptivethreatmodeling/lib/experimental/adaptivethreatmodeling/EndpointCharacteristics.qll

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -618,8 +618,8 @@ private class ModeledDatabaseAccessCharacteristic extends NosqlInjectionSinkEndp
618618
}
619619
}
620620

621-
private class ReceiverIsHTTPRequestExpressionCharacteristic extends NosqlInjectionSinkEndpointFilterCharacteristic {
622-
ReceiverIsHTTPRequestExpressionCharacteristic() { this = "receiver is a HTTP request expression" }
621+
private class ReceiverIsHttpRequestExpressionCharacteristic extends NosqlInjectionSinkEndpointFilterCharacteristic {
622+
ReceiverIsHttpRequestExpressionCharacteristic() { this = "receiver is a HTTP request expression" }
623623

624624
override predicate getEndpoints(DataFlow::Node n) {
625625
exists(DataFlow::CallNode call | n = call.getAnArgument() |
@@ -629,8 +629,8 @@ private class ReceiverIsHTTPRequestExpressionCharacteristic extends NosqlInjecti
629629
}
630630
}
631631

632-
private class ReceiverIsHTTPResponseExpressionCharacteristic extends NosqlInjectionSinkEndpointFilterCharacteristic {
633-
ReceiverIsHTTPResponseExpressionCharacteristic() {
632+
private class ReceiverIsHttpResponseExpressionCharacteristic extends NosqlInjectionSinkEndpointFilterCharacteristic {
633+
ReceiverIsHttpResponseExpressionCharacteristic() {
634634
this = "receiver is a HTTP response expression"
635635
}
636636

@@ -698,8 +698,8 @@ abstract private class SqlInjectionSinkEndpointFilterCharacteristic extends Endp
698698
}
699699
}
700700

701-
private class PreparedSQLStatementCharacteristic extends SqlInjectionSinkEndpointFilterCharacteristic {
702-
PreparedSQLStatementCharacteristic() { this = "prepared SQL statement" }
701+
private class PreparedSqlStatementCharacteristic extends SqlInjectionSinkEndpointFilterCharacteristic {
702+
PreparedSqlStatementCharacteristic() { this = "prepared SQL statement" }
703703

704704
override predicate getEndpoints(DataFlow::Node n) {
705705
exists(DataFlow::CallNode call | n = call.getAnArgument() |
@@ -721,8 +721,8 @@ private class ArrayCreationCharacteristic extends SqlInjectionSinkEndpointFilter
721721
}
722722
}
723723

724-
private class HTMLOrRenderingCharacteristic extends SqlInjectionSinkEndpointFilterCharacteristic {
725-
HTMLOrRenderingCharacteristic() { this = "HTML / rendering" }
724+
private class HtmlOrRenderingCharacteristic extends SqlInjectionSinkEndpointFilterCharacteristic {
725+
HtmlOrRenderingCharacteristic() { this = "HTML / rendering" }
726726

727727
override predicate getEndpoints(DataFlow::Node n) {
728728
exists(DataFlow::CallNode call | n = call.getAnArgument() |

0 commit comments

Comments
 (0)