starred/hickory-dns

Fork 0

mirror of https://github.com/hickory-dns/hickory-dns.git synced 2026-04-25 03:05:51 +03:00

[GH-ISSUE #403] Server hangs on any malformed UDP packet #477

New issue

Closed

opened 2026-03-15 22:43:38 +03:00 by kerem · 8 comments

kerem commented

2026-03-15 22:43:38 +03:00

Owner

Originally created by @krant on GitHub (Apr 17, 2018).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/403

Steps to reproduce:

$ cd server
$ ../target/release/named -c ./tests/named_test_configs/example.toml -z ./tests/named_test_configs/ -p 24141

in another console:

$ dig @127.0.0.1 -p 24141 www.example.com
<works>
$ echo "hello" >/dev/udp/127.0.0.1/24141
$ dig @127.0.0.1 -p 24141 www.example.com
<hangs>

Originally created by @krant on GitHub (Apr 17, 2018). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/403 Steps to reproduce: ``` $ cd server $ ../target/release/named -c ./tests/named_test_configs/example.toml -z ./tests/named_test_configs/ -p 24141 ``` in another console: ``` $ dig @127.0.0.1 -p 24141 www.example.com <works> $ echo "hello" >/dev/udp/127.0.0.1/24141 $ dig @127.0.0.1 -p 24141 www.example.com <hangs> ```

kerem

2026-03-15 22:43:38 +03:00

closed this issue
added the
crate:server

bug:critical
labels

kerem commented

2026-03-15 22:43:54 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 17, 2018):

In your Cargo.lock file, can you grep for tokio-core and it’s associated version?

@bluejekyll commented on GitHub (Apr 17, 2018): In your Cargo.lock file, can you grep for tokio-core and it’s associated version?

kerem commented

2026-03-15 22:43:59 +03:00

Author

Owner

@krant commented on GitHub (Apr 17, 2018):

Sure, it is tokio-core 0.1.16.

@krant commented on GitHub (Apr 17, 2018): Sure, it is `tokio-core 0.1.16`.

kerem commented

2026-03-15 22:44:04 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 17, 2018):

Cool. Thanks. I’ll take a look at this later. I don’t have a lot of negative tests in the library, so this is great help, thanks for the submission.

I really need to put a fuzzer on the library.

@bluejekyll commented on GitHub (Apr 17, 2018): Cool. Thanks. I’ll take a look at this later. I don’t have a lot of negative tests in the library, so this is great help, thanks for the submission. I really need to put a fuzzer on the library.

kerem commented

2026-03-15 22:44:09 +03:00

Author

Owner

@DemiMarie commented on GitHub (Apr 17, 2018):

@bluejekyll Could AFL be used? My understanding is that there is Rust integration somewhere.

@DemiMarie commented on GitHub (Apr 17, 2018): @bluejekyll Could AFL be used? My understanding is that there is Rust integration somewhere.

kerem commented

2026-03-15 22:44:15 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 17, 2018):

There are a few fuzzers available in Rust. I filed this issue for it: https://github.com/bluejekyll/trust-dns/issues/97

it references cargo-fuzz.

@bluejekyll commented on GitHub (Apr 17, 2018): There are a few fuzzers available in Rust. I filed this issue for it: https://github.com/bluejekyll/trust-dns/issues/97 it references `cargo-fuzz`.

kerem commented

2026-03-15 22:44:20 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 18, 2018):

Ok, reproduced this with a test in #407 working on a fix

@bluejekyll commented on GitHub (Apr 18, 2018): Ok, reproduced this with a test in #407 working on a fix

kerem commented

2026-03-15 22:44:25 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 18, 2018):

Thanks for reporting this. There's now a test case for this. Turned out that the error was escaping the UDP handler, causing the UDP socket to be shutdown.

Thanks again for reporting this.

@bluejekyll commented on GitHub (Apr 18, 2018): Thanks for reporting this. There's now a test case for this. Turned out that the error was escaping the UDP handler, causing the UDP socket to be shutdown. Thanks again for reporting this.

kerem commented

2026-03-15 22:44:30 +03:00

Author

Owner

@krant commented on GitHub (Apr 18, 2018):

Thanks for great software!

@krant commented on GitHub (Apr 18, 2018): Thanks for great software!

kerem referenced this issue

2026-03-16 02:08:07 +03:00

[PR #477] [MERGED] client: client_future: fix doc typo #1467