Bug: Undefined behavior in param::hstring(std::wstring_view
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 52/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- cpp
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu từ hàm dựng param::hstring nhận std::wstring_view và lần theo lời gọi đến impl::create_hstring_on_stack(), nơi quyền truy cập ngoài phạm vi được thể hiện. Tái hiện trường hợp bộ đệm có kích thước cố định được mô tả trong issue, sau đó xác minh rằng hàm dựng không còn đọc vượt quá view và thêm coverage hồi quy cho đầu vào không được kết thúc bằng null.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Version
v2.0.250303.1
Summary
The param::hstring constructor that takes a std::wstring_view involves undefined behavior.
A std::wstring_view is not guaranteed to be null-terminated. If it's constructed from a fixed-size buffer, the code from impl::create_hstring_on_stack() will result in UB due to out-of-bounds access:
if (value[length] != 0)
{
abort();
}
In most cases, value[length] will cause a segmentation fault, without reaching the abort() call.
Reproducible example
Expected behavior
No response
Actual behavior
No response
Additional comments
No response
- Ngôn ngữ chính
- C++
- Star
- 1.9k
- Fork
- 281
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/cppwinrt
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
base_macros.h disables warnings without push/pop, leaking them into consumer translation units Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 20/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
Tất cả issue của microsoft/cppwinrt
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
duckdb/duckdb-wasm#2258 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
objectionary/eo-graphs#75 ·
-
Coarray integration tests carry no LABELS, so run_tests.py silently skips them under every backend Đang mởcoarray
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
FISCO-BCOS/FISCO-BCOS#5642 ·