Skip to content

fix: Panic in write_utf8 (assert s.len() < 64)#484

Merged
keepsimple1 merged 3 commits into
mainfrom
write-utf8
Jul 25, 2026
Merged

fix: Panic in write_utf8 (assert s.len() < 64)#484
keepsimple1 merged 3 commits into
mainfrom
write-utf8

Conversation

@keepsimple1

@keepsimple1 keepsimple1 commented Jul 21, 2026

Copy link
Copy Markdown
Owner

This is to fix issue #483

The fix replaces the panic with graceful error handling by threading a Result through the write path:

  • New WriteError enum and WriteResult alias.
  • DnsRecordExt::write, write_question, write_record, write_name, write_utf8 all return WriteResult instead of panicking / returning bool.

@keepsimple1
keepsimple1 merged commit 08362e3 into main Jul 25, 2026
3 checks passed
@keepsimple1
keepsimple1 deleted the write-utf8 branch July 25, 2026 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant