Project

General

Profile

Actions
Copy link

Bug #2207

closed

DNS UDP "Response" parsing recording an incorrect value

Added by Derek Kingsbury about 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Derek Kingsbury
Target version:
4.0.1
Affected Versions:
Effort:
Difficulty:
Label:

Description

Must change
if (f != NULL) {
dns_state->last_req = f->lastts;
}

to:

if (f != NULL) {
        dns_state->last_resp = f->lastts;
    }

The procedure is currently recording the last request (last_req), rather than the last response (last_resp) as it should to be consistent with the Response parsing.

Actions
Copy link
 #1

Updated by Andreas Herz about 7 years ago

Since you found it, would you mind sending a PR for that? See more details at https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing

Actions
Copy link
 #2

Updated by Victor Julien about 7 years ago

  • Status changed from New to Closed
Actions
Copy link
 #3

Updated by Victor Julien about 7 years ago

  • Subject changed from DNS UDP "Respone" parsing recording an incorrect value to DNS UDP "Response" parsing recording an incorrect value
Actions
Copy link

Also available in: Atom PDF