Project

General

Profile

Actions

Security #2902

closed

rust/dhcp: panic in dhcp parser

Added by Jason Ish over 5 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Label:
Git IDs:

41aab88ddc7bb593cf1be58e137af85b8bf553f5

Severity:
Disclosure Date:

Description

From reporter:

==14370== ERROR: libFuzzer: deadly signal
...
/home/sirko/Projects/CI/fuzzing/suricata-fuzzing.2/rust/src/dhcp/parser.rs:126:23
#17 0x56083d83ff5b in suricata::dhcp::parser::parse_option::hab72aeff1560bad1 /home/sirko/Projects/CI/fuzzing/suricata-
fuzzing.2/rust/<::nom::macros::named macros>:38:46
#18 0x56083d80582b in suricata::dhcp::parser::dhcp_parse::h5f41b0fc5736d132 /home/sirko/Projects/CI/fuzzing/suricata-
fuzzing.2/rust/src/dhcp/parser.rs:205:22
#19 0x56083d7e4e8f in suricata::dhcp::dhcp::DHCPState::parse::h7ace958910b14aac /home/sirko/Projects/CI/fuzzing/suricata-
fuzzing.2/rust/src/dhcp/dhcp.rs:146:14
#20 0x56083d72dfbc in rust_fuzzer_test_input /home/sirko/Projects/CI/fuzzing/suricata-fuzzing.2/rust/fuzz/fuzz_targets/
fuzz_dhcp.rs:7:4
#21 0x56083d9b2744 in libfuzzer_sys::test_input_wrap::_$u7b$$u7b$closure$u7d$$u7d$::h29c9181044b7489b
/home/sirko/.cargo/git/checkouts/libfuzzer-sys-e07fde05820d7bc6/4a41319/src/lib.rs:11:8
#22 0x56083d9f984d in std::panicking::try::do_call::hd66afc279650fe66
/rustc/0f88167f89fffe321590c5148f21b7d51d44388d/src/libstd/panicking.rs:293:39
#23 0x56083da0afe8 in __rust_maybe_catch_panic /rustc/0f88167f89fffe321590c5148f21b7d51d44388d/src/libpanic_abort/
lib.rs:29:4
NOTE: libFuzzer has rudimentary signal handlers.
Combine libFuzzer with AddressSanitizer or similar for better crash reports.
SUMMARY: libFuzzer: deadly signal

The error is from an attempt to parse len - 1 bytes without first checking that len is > 0.


Related issues 1 (0 open1 closed)

Copied to Suricata - Security #2947: rust/dhcp: panic in dhcp parser (master)ClosedJason IshActions
Actions #1

Updated by Victor Julien over 5 years ago

  • Copied to Security #2947: rust/dhcp: panic in dhcp parser (master) added
Actions #2

Updated by Victor Julien over 5 years ago

  • Status changed from Assigned to Closed
Actions #3

Updated by Victor Julien over 5 years ago

  • Private changed from Yes to No
Actions #4

Updated by Victor Julien about 4 years ago

  • Tracker changed from Bug to Security
  • Effort deleted (low)
  • Difficulty deleted (low)
  • CVE set to 2019-10052
  • Git IDs updated (diff)
Actions

Also available in: Atom PDF