common: json_to_s64 and json_to_double still accept hex, octal and signs
Maintainer thường phản hồi trong vòng 1 ngà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
- 68/100
Hướng nghiên cứu
Bắt đầu bằng cách tìm json_to_s64() và json_to_double(), sau đó so sánh hành vi phân tích cú pháp của chúng với json_to_u64() và nhánh str_to_u64() nghiêm ngặt được mô tả trong issue. Xác minh rằng số thập lục phân, số bát phân và dấu được ghi rõ bị từ chối, trong khi các số JSON hợp lệ vẫn được chấp nhận, bao gồm thay đổi đã nêu từ "010" thành 10.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
#9150 routed json_to_u64() through the new str_to_u64(), which accepts strict decimal only. json_to_s64() and json_to_double() still use strtoll/strtod with base 0, so they accept hex (0x...), octal, and a leading +/-, none of which are valid JSON numbers. u64 is now asymmetric with the other two.
Note the u64 change already altered one case: "010" used to parse as 8 (base-0 octal) and now parses as 10. Correct per JSON, but worth being aware of when tightening the others.
No urgency, this has been the parsing behaviour for a long time.
- Ngôn ngữ chính
- C
- Star
- 3.1k
- Fork
- 1k
- Merge trung bình
- 3 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 30
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Không có 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 ElementsProject/lightning
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
ElementsProject/lightning#9634 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
plugins/recover: missing comma in nodes_for_gossip concatenates two bootstrap addresses into invalid addressCó thể đã có người làm @bhuvan-somisetty đã nhận 2 ngày trước. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
ElementsProject/lightning#9630 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
common/json_parse: json_to_bitcoin_amount fails to detect overflow and accepts negative/empty inputsCó thể đã có người làm @bhuvan-somisetty đã nhận 7 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
ElementsProject/lightning#9617 ·
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 66/100
ElementsProject/lightning#9616 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/lightning#9593 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của ElementsProject/lightning
Issue tương tự
-
[raylib] update to 6.2Đang mởcategory:port-update
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
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 74/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
Maintainer thường phản hồi trong vòng 8 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
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 90/100
Maintainer thường phản hồi trong vòng 1 ngày