Skip to content
Snippets Groups Projects
Commit e6c09180 authored by Robert Edmonds's avatar Robert Edmonds
Browse files

Revert "mod-dnstap: Fix corner case in "Restore the original query QNAME case""

This reverts commit 55663f09.
parent 4a23a53d
No related merge requests found
......@@ -61,28 +61,24 @@ typedef struct {
bool with_queries;
} dnstap_ctx_t;
static void msg_query_qname_restore(Dnstap__Message *msg,
const knot_dname_t *orig_qname,
const uint16_t qname_size)
static void msg_query_qname_restore(Dnstap__Message *msg, knotd_qdata_t *qdata)
{
if (msg->query_message.data == NULL ||
orig_qname == NULL ||
orig_qname[0] == '\x00')
{
if (msg->query_message.data == NULL) {
return;
}
const knot_dname_t *orig_qname = knotd_qdata_orig_qname(qdata);
if (orig_qname == NULL) {
return;
}
memcpy(msg->query_message.data + KNOT_WIRE_HEADER_SIZE,
orig_qname, qname_size);
orig_qname, qdata->query->qname_size);
}
static void msg_query_qname_case_lower(Dnstap__Message *msg,
const knot_dname_t *orig_qname)
static void msg_query_qname_case_lower(Dnstap__Message *msg)
{
if (msg->query_message.data == NULL ||
orig_qname == NULL ||
orig_qname[0] == '\x00')
{
if (msg->query_message.data == NULL) {
return;
}
......@@ -157,18 +153,12 @@ static knotd_state_t log_message(knotd_state_t state, const knot_pkt_t *pkt,
msg.has_query_message = 1;
}
/*
* Get the original QNAME (if present) in order to write the original
* cased QNAME into the dnstap message.
*/
const knot_dname_t *orig_qname = knotd_qdata_orig_qname(qdata);
/* Pack the message. */
uint8_t *frame = NULL;
size_t size = 0;
msg_query_qname_restore(&msg, orig_qname, qdata->query->qname_size);
msg_query_qname_restore(&msg, qdata);
dt_pack(&dnstap, &frame, &size);
msg_query_qname_case_lower(&msg, orig_qname);
msg_query_qname_case_lower(&msg);
if (frame == NULL) {
return state;
}
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment