mirror of
https://github.com/github/codeql.git
synced 2025-12-17 17:23:36 +01:00
C++: Fix two bugs found by @rdmarsh2
Co-Authored-By: Robert Marsh <rdmarsh2@gmail.com>
This commit is contained in:
@@ -34,6 +34,6 @@ where
|
|||||||
not taintedWithoutGlobals(arg) and
|
not taintedWithoutGlobals(arg) and
|
||||||
taintedWithPath(userValue, arg, sourceNode, sinkNode) and
|
taintedWithPath(userValue, arg, sourceNode, sinkNode) and
|
||||||
isUserInput(userValue, cause)
|
isUserInput(userValue, cause)
|
||||||
select arg,
|
select arg, sourceNode, sinkNode,
|
||||||
"The value of this argument may come from $@ and is being used as a formatting argument to " +
|
"The value of this argument may come from $@ and is being used as a formatting argument to " +
|
||||||
printfFunction, userValue, cause
|
printfFunction, userValue, cause
|
||||||
|
|||||||
@@ -489,7 +489,7 @@ module TaintedWithPath {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private class WrapPathNode extends PathNode, TPathNode {
|
private class WrapPathNode extends PathNode, TWrapPathNode {
|
||||||
DataFlow3::PathNode inner() { this = TWrapPathNode(result) }
|
DataFlow3::PathNode inner() { this = TWrapPathNode(result) }
|
||||||
|
|
||||||
override string toString() { result = this.inner().toString() }
|
override string toString() { result = this.inner().toString() }
|
||||||
|
|||||||
Reference in New Issue
Block a user