Skip to content

[3.0.0 prep] New pcap_* bindings - #403

Open
cicku wants to merge 7 commits into
rust-pcap:mainfrom
cicku:2-new-bindings
Open

[3.0.0 prep] New pcap_* bindings#403
cicku wants to merge 7 commits into
rust-pcap:mainfrom
cicku:2-new-bindings

Conversation

@cicku

@cicku cicku commented Aug 24, 2026

Copy link
Copy Markdown
Contributor
  • pcap_init
  • pcap_dump_ftell
  • pcap_dump_ftell64
  • pcap_dump_file
  • pcap_snapshot

pcap_init fixes #383 and supersedes #384.

Must be merged after #400, and a rebase will be done then.

@cicku
cicku marked this pull request as ready for review September 4, 2026 14:48
@cicku

cicku commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Ready:

Reason of additions:

  • pcap_init: Non-ASCII handling
  • pcap_dump_ftell64/ftell: Size-based file rotation needs
  • pcap_snapshot: For Capture<Offline> the snapshot length comes from the file header and is otherwise unknowable

I'm open to drop pcap_dump_file if you feel like certain low level functions are not needed. Instead one can always flush() and offset().

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.

[Bug] reading pcap files on non UTF-8 Windows system fails due to convert path to str fails

1 participant