suricata
suricata copied to clipboard
smtp: Add 535 Code
Make sure these boxes are signed before submitting your Pull Request -- thank you.
- [ V] I have read the contributing guide lines at https://docs.suricata.io/en/latest/devguide/contributing/contribution-process.html
- [ V] I have signed the Open Information Security Foundation contribution agreement at https://suricata.io/about/contribution-agreement/ (note: this is only required once)
- [ V] I have updated the user guide (in doc/userguide/) to reflect the changes made (if applicable)
Link to redmine ticket:
Describe changes:
- I add SMTP_REPLY_535 in app-layer-smtp.c cause some SMTP server allow multiple AUTH LOGIN. The entire session flow will be interrupted when a AUTH LOGIN failure is encountered if the 535 status code is considered illegal.
NOTE: This PR may contain new authors.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
a37fa62
) 82.15% compared to head (34f6487
) 82.13%. Report is 39 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #10128 +/- ##
==========================================
- Coverage 82.15% 82.13% -0.02%
==========================================
Files 974 974
Lines 271925 271925
==========================================
- Hits 223394 223340 -54
- Misses 48531 48585 +54
Flag | Coverage Δ | |
---|---|---|
fuzzcorpus | 62.89% <ø> (+<0.01%) |
:arrow_up: |
suricata-verify | 61.42% <ø> (-0.03%) |
:arrow_down: |
unittests | 62.85% <ø> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Can you also create a ticket, then reference the ticket in the commit and generally add the description in the PR into the commit message? Additionally, a test case for Suricata-Verify would be great. Thanks!
Can you also create a ticket, then reference the ticket in the commit and generally add the description in the PR into the commit message? Additionally, a test case for Suricata-Verify would be great. Thanks!
Sorry, I cant create a account in Suricata issues website (I have not received the registration code in my Outlook email when I created the account) for creating a ticket which as the basis for this commit.
I think you can complete all the status codes that an SMTP server might support, to avoid interruption of stream.
I created https://redmine.openinfosecfoundation.org/issues/6821
Superseded by https://github.com/OISF/suricata/pull/11193