[GH-ISSUE #2985] Fix SIG(0) verification to use original message encoding #1103

Open
opened 2026-03-16 01:36:57 +03:00 by kerem · 2 comments
Owner

Originally created by @divergentdave on GitHub (May 9, 2025).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/2985

PR #2982 fixes TSIG verification to use the original bytes of the message, off the wire, instead of decoding and re-encoding. We should fix SIG(0) verification in a similar manner. This will be a bit more involved, as we'll at least have to change a couple method signatures, and likely fix up the header and truncate the SIG(0) RR by operating on a buffer directly.

Originally created by @divergentdave on GitHub (May 9, 2025). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/2985 PR #2982 fixes TSIG verification to use the original bytes of the message, off the wire, instead of decoding and re-encoding. We should fix SIG(0) verification in a similar manner. This will be a bit more involved, as we'll at least have to change a couple method signatures, and likely fix up the header and truncate the SIG(0) RR by operating on a buffer directly.
Author
Owner

@djc commented on GitHub (May 10, 2025):

Yes, I started looking at this and decided it looked a bit strange...

<!-- gh-comment-id:2868490945 --> @djc commented on GitHub (May 10, 2025): Yes, I started looking at this and decided it looked a bit strange...
Author
Owner

@djc commented on GitHub (May 12, 2025):

Some initial refactoring in this direction in #2986.

<!-- gh-comment-id:2872610801 --> @djc commented on GitHub (May 12, 2025): Some initial refactoring in this direction in #2986.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hickory-dns#1103
No description provided.