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

Dns invalid additionals 7228 v4 #11794

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Commits on Sep 16, 2024

  1. dns: improved handling of corrupt additionals

    Ticket: 7228
    
    That means log the rest of queries and answers, even if the
    final field additionals is corrupt.
    Set an event in this case.
    catenacyber committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    153abc4 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2024

  1. dns: make dns probing function stricter

    To accomodate cases found in QA.
    The case looks like a CnC channel on port 53, but not using DNS.
    
    First request has a very big record length, but all the header
    fields indicating numbers are 0.
    
    First reply has a very big record length as well, there is one
    non-zero field for number, but the flag indicates a request...
    catenacyber committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    e898e77 View commit details
    Browse the repository at this point in the history