Skip to content

Commit af6588e

Browse files
Replacing tabs for spaces
1 parent d4825af commit af6588e

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

csharp/ql/src/Likely Bugs/ICryptoTransform.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ class UsesICryptoTransform extends Class {
2828
class LambdaCapturingICryptoTransformSource extends DataFlow::Node {
2929
LambdaCapturingICryptoTransformSource() {
3030
exists( LambdaExpr l, LocalScopeVariable lsvar, UsesICryptoTransform ict |
31-
l = this.asExpr() |
32-
ict = lsvar.getType()
33-
and lsvar.getACapturingCallable() = l
31+
l = this.asExpr() |
32+
ict = lsvar.getType()
33+
and lsvar.getACapturingCallable() = l
3434
)
3535
}
3636
}

csharp/ql/src/Likely Bugs/ThreadUnsafeICryptoTransformLambda.ql

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ class NotThreadSafeCryptoUsageIntoStartingCallingConfig extends TaintTracking::C
2727

2828
override predicate isSink(DataFlow::Node sink) {
2929
exists( DelegateCreation dc, Expr e |
30-
e = sink.asExpr() |
31-
dc.getArgument() = e
32-
and dc.getType().getName().matches("%Start")
30+
e = sink.asExpr() |
31+
dc.getArgument() = e
32+
and dc.getType().getName().matches("%Start")
3333
)
3434
}
3535
}
@@ -42,18 +42,18 @@ class NotThreadSafeCryptoUsageIntoParallelInvokeConfig extends TaintTracking::Co
4242
}
4343

4444
override predicate isSink(DataFlow::Node sink) {
45-
sink instanceof ParallelSink
45+
sink instanceof ParallelSink
4646
}
4747
}
4848

4949
from Expr e, string m, LambdaExpr l
5050
where
5151
exists( NotThreadSafeCryptoUsageIntoParallelInvokeConfig config |
52-
config.hasFlow(DataFlow::exprNode(l), DataFlow::exprNode(e))
53-
and m = "A $@ seems to be used to start a new thread using System.Threading.Tasks.Parallel.Invoke, and is capturing a local variable that either implements 'System.Security.Cryptography.ICryptoTransform' or has a field of this type."
52+
config.hasFlow(DataFlow::exprNode(l), DataFlow::exprNode(e))
53+
and m = "A $@ seems to be used to start a new thread using System.Threading.Tasks.Parallel.Invoke, and is capturing a local variable that either implements 'System.Security.Cryptography.ICryptoTransform' or has a field of this type."
5454
)
5555
or exists ( NotThreadSafeCryptoUsageIntoStartingCallingConfig config |
56-
config.hasFlow(DataFlow::exprNode(l), DataFlow::exprNode(e))
57-
and m = "A $@ seems to be used to start a new thread is capturing a local variable that either implements 'System.Security.Cryptography.ICryptoTransform' or has a field of this type."
56+
config.hasFlow(DataFlow::exprNode(l), DataFlow::exprNode(e))
57+
and m = "A $@ seems to be used to start a new thread is capturing a local variable that either implements 'System.Security.Cryptography.ICryptoTransform' or has a field of this type."
5858
)
5959
select e, m, l, "lambda expression"

0 commit comments

Comments
 (0)