mirror of
https://github.com/github/codeql.git
synced 2026-05-04 13:15:21 +02:00
Move contextual queries from src to lib
With this change, users are now able to run View AST command in vscode within vscode workspaces that do not include the core libraries. The relevant core library only needs to be installed in the package cache.
This commit is contained in:
22
cpp/ql/lib/IDEContextual.qll
Normal file
22
cpp/ql/lib/IDEContextual.qll
Normal file
@@ -0,0 +1,22 @@
|
||||
/**
|
||||
* Provides shared predicates related to contextual queries in the code viewer.
|
||||
*/
|
||||
|
||||
import semmle.files.FileSystem
|
||||
|
||||
/**
|
||||
* Returns the `File` matching the given source file name as encoded by the VS
|
||||
* Code extension.
|
||||
*/
|
||||
cached
|
||||
File getFileBySourceArchiveName(string name) {
|
||||
// The name provided for a file in the source archive by the VS Code extension
|
||||
// has some differences from the absolute path in the database:
|
||||
// 1. colons are replaced by underscores
|
||||
// 2. there's a leading slash, even for Windows paths: "C:/foo/bar" ->
|
||||
// "/C_/foo/bar"
|
||||
// 3. double slashes in UNC prefixes are replaced with a single slash
|
||||
// We can handle 2 and 3 together by unconditionally adding a leading slash
|
||||
// before replacing double slashes.
|
||||
name = ("/" + result.getAbsolutePath().replaceAll(":", "_")).replaceAll("//", "/")
|
||||
}
|
||||
210
cpp/ql/lib/definitions.qll
Normal file
210
cpp/ql/lib/definitions.qll
Normal file
@@ -0,0 +1,210 @@
|
||||
/**
|
||||
* Provides classes and predicates related to jump-to-definition links
|
||||
* in the code viewer.
|
||||
*/
|
||||
|
||||
import cpp
|
||||
import IDEContextual
|
||||
|
||||
/**
|
||||
* Any element that might be the source or target of a jump-to-definition
|
||||
* link.
|
||||
*
|
||||
* In some cases it is preferable to modify locations (the
|
||||
* `hasLocationInfo()` predicate) so that they are short, and
|
||||
* non-overlapping with other locations that might be highlighted in
|
||||
* the LGTM interface.
|
||||
*
|
||||
* We need to give locations that may not be in the database, so
|
||||
* we use `hasLocationInfo()` rather than `getLocation()`.
|
||||
*/
|
||||
class Top extends Element {
|
||||
/**
|
||||
* Holds if this element is at the specified location.
|
||||
* The location spans column `startcolumn` of line `startline` to
|
||||
* column `endcolumn` of line `endline` in file `filepath`.
|
||||
* For more information, see
|
||||
* [Locations](https://codeql.github.com/docs/writing-codeql-queries/providing-locations-in-codeql-queries/).
|
||||
*/
|
||||
pragma[noopt]
|
||||
final predicate hasLocationInfo(
|
||||
string filepath, int startline, int startcolumn, int endline, int endcolumn
|
||||
) {
|
||||
interestingElement(this) and
|
||||
not this instanceof MacroAccess and
|
||||
not this instanceof Include and
|
||||
exists(Location l |
|
||||
l = this.getLocation() and
|
||||
l.hasLocationInfo(filepath, startline, startcolumn, endline, endcolumn)
|
||||
)
|
||||
or
|
||||
// This has a location that covers only the name of the accessed
|
||||
// macro, not its arguments (which are included by `MacroAccess`'s
|
||||
// `getLocation()`).
|
||||
exists(Location l, MacroAccess ma |
|
||||
ma instanceof MacroAccess and
|
||||
ma = this and
|
||||
l = ma.getLocation() and
|
||||
l.hasLocationInfo(filepath, startline, startcolumn, _, _) and
|
||||
endline = startline and
|
||||
exists(string macroName, int nameLength, int nameLengthMinusOne |
|
||||
macroName = ma.getMacroName() and
|
||||
nameLength = macroName.length() and
|
||||
nameLengthMinusOne = nameLength - 1 and
|
||||
endcolumn = startcolumn + nameLengthMinusOne
|
||||
)
|
||||
)
|
||||
or
|
||||
hasLocationInfo_Include(this, filepath, startline, startcolumn, endline, endcolumn)
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* An `Include` with a `hasLocationInfo` predicate.
|
||||
*
|
||||
* This has a location that covers only the name of the included
|
||||
* file, not the `#include` text or whitespace before it.
|
||||
*/
|
||||
predicate hasLocationInfo_Include(Include i, string path, int sl, int sc, int el, int ec) {
|
||||
exists(Location l |
|
||||
l = i.getLocation() and
|
||||
path = l.getFile().getAbsolutePath() and
|
||||
sl = l.getEndLine() and
|
||||
sc = l.getEndColumn() + 1 - i.getIncludeText().length() and
|
||||
el = l.getEndLine() and
|
||||
ec = l.getEndColumn()
|
||||
)
|
||||
}
|
||||
|
||||
/** Holds if `e` is a source or a target of jump-to-definition. */
|
||||
predicate interestingElement(Element e) {
|
||||
exists(definitionOf(e, _))
|
||||
or
|
||||
e = definitionOf(_, _)
|
||||
}
|
||||
|
||||
/**
|
||||
* Holds if `f`, `line`, `column` indicate the start character
|
||||
* of `cc`.
|
||||
*/
|
||||
private predicate constructorCallStartLoc(ConstructorCall cc, File f, int line, int column) {
|
||||
exists(Location l |
|
||||
l = cc.getLocation() and
|
||||
l.getFile() = f and
|
||||
l.getStartLine() = line and
|
||||
l.getStartColumn() = column
|
||||
)
|
||||
}
|
||||
|
||||
/**
|
||||
* Holds if `f`, `line`, `column` indicate the start character
|
||||
* of `tm`, which mentions `t`. Type mentions for instantiations
|
||||
* are filtered out.
|
||||
*/
|
||||
private predicate typeMentionStartLoc(TypeMention tm, Type t, File f, int line, int column) {
|
||||
exists(Location l |
|
||||
l = tm.getLocation() and
|
||||
l.getFile() = f and
|
||||
l.getStartLine() = line and
|
||||
l.getStartColumn() = column
|
||||
) and
|
||||
t = tm.getMentionedType() and
|
||||
not t instanceof ClassTemplateInstantiation
|
||||
}
|
||||
|
||||
/**
|
||||
* Holds if `cc` and `tm` begin at the same character.
|
||||
*/
|
||||
cached
|
||||
private predicate constructorCallTypeMention(ConstructorCall cc, TypeMention tm) {
|
||||
exists(File f, int line, int column |
|
||||
constructorCallStartLoc(cc, f, line, column) and
|
||||
typeMentionStartLoc(tm, _, f, line, column)
|
||||
)
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets an element, of kind `kind`, that element `e` uses, if any.
|
||||
* Attention: This predicate yields multiple definitions for a single location.
|
||||
*
|
||||
* The `kind` is a string representing what kind of use it is:
|
||||
* - `"M"` for function and method calls
|
||||
* - `"T"` for uses of types
|
||||
* - `"V"` for variable accesses
|
||||
* - `"X"` for macro accesses
|
||||
* - `"I"` for import / include directives
|
||||
*/
|
||||
cached
|
||||
Top definitionOf(Top e, string kind) {
|
||||
(
|
||||
// call -> function called
|
||||
kind = "M" and
|
||||
result = e.(Call).getTarget() and
|
||||
not e.(Expr).isCompilerGenerated() and
|
||||
not e instanceof ConstructorCall // handled elsewhere
|
||||
or
|
||||
// access -> function, variable or enum constant accessed
|
||||
kind = "V" and
|
||||
result = e.(Access).getTarget() and
|
||||
not e.(Expr).isCompilerGenerated()
|
||||
or
|
||||
// macro access -> macro accessed
|
||||
kind = "X" and
|
||||
result = e.(MacroAccess).getMacro()
|
||||
or
|
||||
// type mention -> type
|
||||
kind = "T" and
|
||||
e.(TypeMention).getMentionedType() = result and
|
||||
not constructorCallTypeMention(_, e) and // handled elsewhere
|
||||
// Multiple type mentions can be generated when a typedef is used, and
|
||||
// in such cases we want to exclude all but the originating typedef.
|
||||
not exists(Type secondary |
|
||||
exists(TypeMention tm, File f, int startline, int startcol |
|
||||
typeMentionStartLoc(e, result, f, startline, startcol) and
|
||||
typeMentionStartLoc(tm, secondary, f, startline, startcol) and
|
||||
(
|
||||
result = secondary.(TypedefType).getBaseType() or
|
||||
result = secondary.(TypedefType).getBaseType().(SpecifiedType).getBaseType()
|
||||
)
|
||||
)
|
||||
)
|
||||
or
|
||||
// constructor call -> function called
|
||||
// - but only if there is a corresponding type mention, since
|
||||
// we don't want links for implicit conversions.
|
||||
// - using the location of the type mention, since it's
|
||||
// tighter that the location of the function call.
|
||||
kind = "M" and
|
||||
exists(ConstructorCall cc |
|
||||
constructorCallTypeMention(cc, e) and
|
||||
result = cc.getTarget()
|
||||
)
|
||||
or
|
||||
// include -> included file
|
||||
kind = "I" and
|
||||
result = e.(Include).getIncludedFile() and
|
||||
// exclude `#include` directives containing macros
|
||||
not exists(MacroInvocation mi, Location l1, Location l2 |
|
||||
l1 = e.(Include).getLocation() and
|
||||
l2 = mi.getLocation() and
|
||||
l1.getContainer() = l2.getContainer() and
|
||||
l1.getStartLine() = l2.getStartLine()
|
||||
// (an #include directive must be always on it's own line)
|
||||
)
|
||||
) and
|
||||
(
|
||||
// exclude things inside macro invocations, as they will overlap
|
||||
// with the macro invocation.
|
||||
not e.(Element).isInMacroExpansion() and
|
||||
// exclude nested macro invocations, as they will overlap with
|
||||
// the top macro invocation.
|
||||
not exists(e.(MacroAccess).getParentInvocation())
|
||||
) and
|
||||
// Some entities have many locations. This can arise for an external
|
||||
// function that is frequently declared but not defined, or perhaps
|
||||
// for a struct type that is declared in many places. Rather than
|
||||
// letting the result set explode, we just exclude results that are
|
||||
// "too ambiguous" -- we could also arbitrarily pick one location
|
||||
// later on.
|
||||
strictcount(result.getLocation()) < 10
|
||||
}
|
||||
16
cpp/ql/lib/localDefinitions.ql
Normal file
16
cpp/ql/lib/localDefinitions.ql
Normal file
@@ -0,0 +1,16 @@
|
||||
/**
|
||||
* @name Jump-to-definition links
|
||||
* @description Generates use-definition pairs that provide the data
|
||||
* for jump-to-definition in the code viewer of VSCode.
|
||||
* @kind definitions
|
||||
* @id cpp/ide-jump-to-definition
|
||||
* @tags ide-contextual-queries/local-definitions
|
||||
*/
|
||||
|
||||
import definitions
|
||||
|
||||
external string selectedSourceFile();
|
||||
|
||||
from Top e, Top def, string kind
|
||||
where def = definitionOf(e, kind) and e.getFile() = getFileBySourceArchiveName(selectedSourceFile())
|
||||
select e, def, kind
|
||||
17
cpp/ql/lib/localReferences.ql
Normal file
17
cpp/ql/lib/localReferences.ql
Normal file
@@ -0,0 +1,17 @@
|
||||
/**
|
||||
* @name Find-references links
|
||||
* @description Generates use-definition pairs that provide the data
|
||||
* for find-references in the code viewer of VSCode.
|
||||
* @kind definitions
|
||||
* @id cpp/ide-find-references
|
||||
* @tags ide-contextual-queries/local-references
|
||||
*/
|
||||
|
||||
import definitions
|
||||
|
||||
external string selectedSourceFile();
|
||||
|
||||
from Top e, Top def, string kind
|
||||
where
|
||||
def = definitionOf(e, kind) and def.getFile() = getFileBySourceArchiveName(selectedSourceFile())
|
||||
select e, def, kind
|
||||
27
cpp/ql/lib/printAst.ql
Normal file
27
cpp/ql/lib/printAst.ql
Normal file
@@ -0,0 +1,27 @@
|
||||
/**
|
||||
* @name Print AST
|
||||
* @description Outputs a representation of a file's Abstract Syntax Tree. This
|
||||
* query is used by the VS Code extension.
|
||||
* @id cpp/print-ast
|
||||
* @kind graph
|
||||
* @tags ide-contextual-queries/print-ast
|
||||
*/
|
||||
|
||||
import cpp
|
||||
import semmle.code.cpp.PrintAST
|
||||
import definitions
|
||||
|
||||
/**
|
||||
* The source file to generate an AST from.
|
||||
*/
|
||||
external string selectedSourceFile();
|
||||
|
||||
class Cfg extends PrintAstConfiguration {
|
||||
/**
|
||||
* Holds if the AST for `func` should be printed.
|
||||
* Print All functions from the selected file.
|
||||
*/
|
||||
override predicate shouldPrintFunction(Function func) {
|
||||
func.getFile() = getFileBySourceArchiveName(selectedSourceFile())
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user