[Security] tarfile.extractall without member validation in src/crate/testing/layer.py
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 68/100
Hướng nghiên cứu
Bắt đầu trong src/crate/testing/layer.py và kiểm tra lệnh gọi tarfile.extractall() cùng các nơi gọi nó. Xem xét logic safe_extract được đề xuất và xác minh rằng các thành viên của archive không thể ghi ra ngoài thư mục đích. Hoàn tất nghĩa là lỗ hổng Bandit B202 đã được xử lý mà không làm hỏng hành vi trích xuất layer hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Severity: HIGH (Bandit B202)
File: src/crate/testing/layer.py
Vulnerability
tarfile.extractall() without member validation allows path traversal (zip slip). A malicious archive can write files outside the target directory.
Fix
import os
SAFE_ID = __import__("re").compile(r"^[a-zA-Z0-9_.-]+$")
def _is_within_directory(directory, target):
abs_directory = os.path.realpath(directory)
abs_target = os.path.realpath(target)
return abs_target.startswith(abs_directory + os.sep) or abs_target == abs_directory
def safe_extract(tar, path=".", members=None, *, numeric_owner=False):
for member in tar.getmembers():
member_path = os.path.join(path, member.name)
if not _is_within_directory(path, member_path):
raise Exception(f"Path traversal in tar: {member.name}")
tar.extractall(path, members, numeric_owner=numeric_owner)
References
- CWE-22: Path Traversal
- Bandit B202
- OWASP: Zip Slip
- Ngôn ngữ chính
- Python
- Star
- 85
- Fork
- 34
- Merge trung bình
- 3 ngày 1 giờ
- Pull request đã merge (30 ngày)
- 4
Chuẩn bị môi trường
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 crate/crate-python
-
Allow `connect()` (again) while the server is not respondingCó thể đã có người làm @florinutz đã nhận 8 ngày trước. Đang mởenhancement
crate/crate-python#833 · 1 người được giao ·
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
crate/crate-python#826 · 1 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
crate/crate-python#751 · 7 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
crate/crate-python#729 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
crate/crate-python#627 ·
Tất cả issue của crate/crate-python
Issue tương tự
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
kristofdegrave/homeassistant-smart-charging#1413 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
nasa/earthdata-varinfo#113 ·
-
curriculum documentation quality
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
githubnext/gh-aw-workshop#3849 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày