Unverified sandbox items from the Oct 11 audit: proxy checks only the TLS name, and stacked drive-letter definitions

Đã đóng
  • Có gì không ổn?
#293 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
38/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
desktop, security

Hướng nghiên cứu

Read src/main/tools/net_proxy.ts:222-230 and native/sandbox-helper/src/win.rs, then verify both reported behaviors: the proxy's handling of CONNECT and TLS ClientHello extensions, and how QueryDosDeviceW definitions are checked. The issue proposes documenting the proxy limit or refusing ECH, and comparing only the first drive mapping entry; completion means both findings are verified and the chosen fixes prevent the described behavior.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

priority: low security severity: low

Found in the Oct 11 audit (at c5f5641). Both need verification.

  1. The allow-list proxy checks only the TLS server name (src/main/tools/net_proxy.ts:222-230). A CONNECT to an allowed host on a shared CDN can send Host: other-site inside TLS (domain fronting), and Encrypted Client Hello hides the real name behind the CDN's public name. Fix: document the limit, or refuse a ClientHello that carries an ECH extension (0xfe0d).
  2. DriveMapping::is_live accepts any entry of a drive letter's definition stack (native/sandbox-helper/src/win.rs, added in 68dd655). QueryDosDeviceW returns every stacked definition, and .any(...) accepts the expected target even when another definition sits on top, so ProjectDrive::define skips DefineDosDeviceW and the run starts in the wrong folder. Correctness only: access is still governed by ACLs. Fix: compare only the first entry.
Ngôn ngữ chính
TypeScript
Star
2
Fork
2
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.