mrva/vscode-codeql
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Exclude methods from anonymous classes in framework mode

Browse Source
This commit is contained in:
Koen Vlaswinkel
2023-06-23 13:45:06 +02:00
parent 4f367119cb
commit 6e16f826fb
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
extensions/ql-vscode/src/data-extensions-editor/queries/csharp.ts
View File

@@ -37,6 +37,7 @@ select usage, apiName, supported.toString(), "supported", api.getFile().getBaseN
*/ */
private import csharp private import csharp
private import cil
private import dotnet private import dotnet
private import semmle.code.csharp.dispatch.Dispatch private import semmle.code.csharp.dispatch.Dispatch
private import semmle.code.csharp.dataflow.ExternalFlow private import semmle.code.csharp.dataflow.ExternalFlow
@@ -66,11 +67,12 @@ class TestLibrary extends RefType {
/** Holds if the given callable is not worth supporting. */ /** Holds if the given callable is not worth supporting. */
private predicate isUninteresting(DotNet::Callable c) { private predicate isUninteresting(DotNet::Callable c) {
c.getDeclaringType() instanceof TestLibrary or c.getDeclaringType() instanceof TestLibrary or
c.(Constructor).isParameterless() c.(Constructor).isParameterless() or
c.getDeclaringType() instanceof AnonymousClass
} }
class PublicMethod extends DotNet::Member { class PublicMethod extends DotNet::Member {
PublicMethod() { this.isPublic() and not isUninteresting(this) and exists(this.(DotNet::Member)) } PublicMethod() { this.isPublic() and not isUninteresting(this) }
/** /**
* Gets the unbound type, name and parameter types of this API. * Gets the unbound type, name and parameter types of this API.

5
extensions/ql-vscode/src/data-extensions-editor/queries/java.ts
View File

@@ -50,10 +50,11 @@ private import semmle.code.java.dataflow.internal.ModelExclusions
/** Holds if the given callable is not worth supporting. */ /** Holds if the given callable is not worth supporting. */
private predicate isUninteresting(Callable c) { private predicate isUninteresting(Callable c) {
c.getDeclaringType() instanceof TestLibrary or c.getDeclaringType() instanceof TestLibrary or
c.(Constructor).isParameterless() c.(Constructor).isParameterless() or
c.getDeclaringType() instanceof AnonymousClass
} }
class PublicMethod extends Callable { class PublicMethod extends Method {
PublicMethod() { this.isPublic() and not isUninteresting(this) } PublicMethod() { this.isPublic() and not isUninteresting(this) }
/** /**