[C++] Assigning to function pointer in a function appears to defeat dispatch analysis
- Ngôn ngữ chính
- CodeQL
- Star
- 10.1k
- Fork
- 2.1k
- Merge trung bình
- 2 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 141
Mô tả
Assigning to a function pointer in a function appears to defeat the points-to analysis used to resolve the dispatch of function pointers.
In the following example I would have expected to see two flows from `source()` to `target()` but only the second one from the direct assignment of the function pointer is reported.
```cpp
int source()
{
return 2;
}
int a_function()
{
return source();
}
int target(int source)
{
return source;
}
void set(int (**ptr)(), int (*ptr2)())
{
*ptr = ptr2;
}
int main(int argv, char **argc)
{
int (*fptr)();
set(&fptr, a_function);
target(fptr()); // not detected as source
fptr = a_function;
target(fptr()); // detected
return 0;
}
```
This is the complete query
```ql
import cpp
import semmle.code.cpp.dataflow.new.TaintTracking
module SourceSinkCallConfig implements DataFlow::ConfigSig {
predicate isSource(DataFlow::Node source) {
source.asExpr().(FunctionCall).getTarget().getName() = "source"
}
predicate isSink(DataFlow::Node sink) {
exists(Call call |
call.getTarget().getName() = "target" and
call.getArgument(0) = sink.asExpr()
)
}
}
module SourceSinkCallTaint = TaintTracking::Global;
from DataFlow::Node source, DataFlow::Node sink, int source_line, int sink_line
where
SourceSinkCallTaint::flow(source, sink) and
source_line = source.getLocation().getStartLine() and
sink_line = sink.getLocation().getStartLine()
select source, source_line, sink, sink_line
```
This is the output. I would have expected to also see a flow to line 28.
```
| source | source_line | sink | sink_line |
+----------------+-------------+--------------------+-----------+
| call to source | 9 | call to expression | 32 |
```
CodeQL version: 2.19.3
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách chạy truy vấn đầy đủ từ issue trên ví dụ C++ được cung cấp bằng CodeQL 2.19.3. Đọc hành vi về luồng dữ liệu và Points-to của C++ đằng sau thư viện TaintTracking đã được import, sau đó xác minh rằng phép gán gián tiếp thông qua set() được phân giải. Được xem là hoàn tất khi truy vấn báo cáo luồng bổ sung dự kiến đến dòng 28.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- devtools
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 42/100