[GH-ISSUE #1709] I found a recursion which may cause stack overflow error #740

New issue

Open

opened 2026-03-16 00:05:30 +03:00 by kerem · 1 comment

kerem commented

2026-03-16 00:05:30 +03:00

Owner

Originally created by @Ren-ZY on GitHub (May 21, 2022).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/1709

The function read_inner at crates/proto/src/rr/domain/name.rs: 981:1: 981:93 is a recursive function.
Maybe Some special input can make it recursive deeply and cause stack overflow?

For example:
let btyes = vec![..............];
let mut d = BinDecoder::new(&bytes);
Name::read(&mut d);

Originally created by @Ren-ZY on GitHub (May 21, 2022). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/1709 The function *read_inner* at crates/proto/src/rr/domain/name.rs: 981:1: 981:93 is a recursive function. Maybe Some special input can make it recursive deeply and cause stack overflow? For example: let btyes = vec![..............]; let mut d = BinDecoder::new(&bytes); Name::read(&mut d);

kerem added the

question

label

2026-03-16 00:05:30 +03:00

kerem commented

2026-03-16 00:05:41 +03:00

Author

Owner

@bluejekyll commented on GitHub (May 22, 2022):

Have you been able to create a stack overflow? The read_inner recursion has a requirement that it must point to an earlier section of the message for the compressed label. This bounds the read_inner in the sense that it can't infinitely recurse, eventually it would stop at the first index of the message.

@bluejekyll commented on GitHub (May 22, 2022): Have you been able to create a stack overflow? The `read_inner` recursion has a requirement that it must point to an earlier section of the message for the compressed label. This bounds the `read_inner` in the sense that it can't infinitely recurse, eventually it would stop at the first index of the message.

kerem referenced this issue

2026-03-16 02:17:36 +03:00

[PR #740] [CLOSED] Bump serde_derive from 1.0.89 to 1.0.90 #1644