hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-16 16:53:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

Python: Print file path when logging context errors

Browse Source 
This makes it _much_ easier to find the offending bit of syntax.
This commit is contained in:
Taus
2025-02-05 13:13:39 +00:00
parent d7117ef3e5
commit 60d97e0e16
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
python/extractor/semmle/python/parser/tsg_parser.py
View File

@@ -196,13 +196,13 @@ def resolve_node_id(id, node_attr):
id = node_attr[id]["_skip_to"].id
return id
def get_context(id, node_attr, logger):
def get_context(id, node_attr, path, logger):
"""Gets the context of the node with the given `id`. This is either whatever is stored in the
`ctx` attribute of the node, or the result of dereferencing a sequence of `_inherited_ctx` attributes."""
while "ctx" not in node_attr[id]:
if "_inherited_ctx" not in node_attr[id]:
logger.error("No context for node {} with attributes {}\n".format(id, node_attr[id]))
logger.error("No context for node {} in file {} with attributes {}\n".format(id, path, node_attr[id]))
# A missing context is most likely to be a "load", so return that.
return ast.Load()
id = node_attr[id]["_inherited_ctx"].id
@@ -344,7 +344,7 @@ def parse(path, logger):
# Set up context information, if any
if "ctx" in expected_fields:
node.ctx = get_context(id, node_attr, logger)
node.ctx = get_context(id, node_attr, path, logger)
# Set the fields.
for field, val in attrs.items():
if field.startswith("_"): continue