github / github/codeql

[Python] General issue: Wrong domination is reported in control flow analysis

未关闭
#20,551 1 条评论 0 个 reaction 已指派 1 人 已被 @tausbn 认领 在 GitHub 查看
question
主要语言
CodeQL
星标
10.1k
派生
2.1k
平均合并
2 天 15 小时
30 天内合并 PR
141

描述

**Description of the issue**
I have a python code and ql as follows:
``` python
def EXEC():
raise Exception("Test exception")

def API():
pass

def DUMMY():
pass

try:
EXEC()
API()
except Exception as e:
raise(e)
finally:
DUMMY()
```
``` ql
/**
* @kind problem
* @problem.severity warning
* @id python/find-if
*/

import python

predicate isCall(Call call, string name) {
exists (Name n | call.getFunc() = n and n.getId() = name)
}

from Call callAPI, Call callDummy
where
isCall(callAPI, "API") and isCall(callDummy, "DUMMY") and
not exists (ControlFlowNode nAPI, ControlFlowNode nDummy |
nAPI = callAPI.getAFlowNode() and nDummy = callDummy.getAFlowNode() and
nAPI.dominates(nDummy))
select callAPI, callAPI.toString()
```

The ql code simply check if the call to `API()` does not dominate the call to `DUMMY()`
**Expected Behaviour**: Call to `API()` is reported because `API()` does not dominate `DUMMY()`.
**Actual Result**: nothing is reported

CodeQL version: `CodeQL command-line toolchain release 2.23.1.`

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。