Skip to content

Commit 0805b49

Browse files
Update csharp/ql/src/experimental/Security Features/JsonWebTokenHandler/JsonWebTokenHandlerLib.qll
Co-authored-by: Tom Hvitved <hvitved@github.com>
1 parent 6070e4f commit 0805b49

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

csharp/ql/src/experimental/Security Features/JsonWebTokenHandler/JsonWebTokenHandlerLib.qll

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -246,17 +246,14 @@ private class CallableReturnsStringAndArg0IsString extends Callable {
246246
*/
247247
class CallableAlwaysReturnsParameter0 extends CallableReturnsStringAndArg0IsString {
248248
CallableAlwaysReturnsParameter0() {
249-
forall(ReturnStmt rs | rs.getEnclosingCallable() = this |
250-
rs.getExpr() = this.getParameter(0).getAnAccess()
251-
) and
252-
exists(ReturnStmt rs | rs.getEnclosingCallable() = this)
253-
or
254-
exists(AnonymousFunctionExpr le, Call call, CallableAlwaysReturnsParameter0 cat | this = le |
255-
call = le.getExpressionBody() and
256-
cat.getACall() = call
249+
forex(Expr ret | this.canReturn(ret) |
250+
ret = this.getParameter(0).getAnAccess()
251+
or
252+
exists(CallableAlwaysReturnsParameter0 c |
253+
ret = c.getACall() and
254+
ret.(Call).getArgument(0) = this.getParameter(0).getAnAccess()
255+
)
257256
)
258-
or
259-
this.getBody() = this.getParameter(0).getAnAccess()
260257
}
261258
}
262259

0 commit comments

Comments
 (0)