Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Next/20210111/v6 #5718

Closed
wants to merge 10 commits into from
Closed

Conversation

victorjulien
Copy link
Member

#5704 #5715 #5712
some cherry-picks from #5716
ftp-data inspection fixes

victorjulien and others added 10 commits January 12, 2021 08:13
The cbindgen generated header should not include rust.h as
rust.h already includes the generated binding.

Fixup C source code that only pulled the generated include, it
should instead pull in "rust.h" which includes the generated
binding plus other misc. stuff.
The default Python on MacOS is Python 2. Suricata-Verify now
depends on Python 3, so install it with Brew.
This commit improves TFTP parsing by ensuring the mode and opcode are
valid.
This commit adds test cases for the TFTP parser.
@victorjulien victorjulien requested review from jasonish, norg and a team as code owners January 12, 2021 16:38
@victorjulien victorjulien mentioned this pull request Jan 12, 2021
@victorjulien
Copy link
Member Author

Replaced by #5720

@victorjulien victorjulien deleted the next/20210111/v6 branch January 26, 2021 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants