Skip to content

Commit

Permalink
resolved: actually check authenticated flag of SOA transaction
Browse files Browse the repository at this point in the history
Fixes #25676

(cherry picked from commit 3b4cc14)
  • Loading branch information
msekletar authored and bluca committed Dec 22, 2023
1 parent a1597f9 commit 6da5ca9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/resolve/resolved-dns-transaction.c
Original file line number Diff line number Diff line change
Expand Up @@ -2808,7 +2808,7 @@ static int dns_transaction_requires_rrsig(DnsTransaction *t, DnsResourceRecord *
if (r == 0)
continue;

return FLAGS_SET(t->answer_query_flags, SD_RESOLVED_AUTHENTICATED);
return FLAGS_SET(dt->answer_query_flags, SD_RESOLVED_AUTHENTICATED);
}

return true;
Expand All @@ -2835,7 +2835,7 @@ static int dns_transaction_requires_rrsig(DnsTransaction *t, DnsResourceRecord *
/* We found the transaction that was supposed to find the SOA RR for us. It was
* successful, but found no RR for us. This means we are not at a zone cut. In this
* case, we require authentication if the SOA lookup was authenticated too. */
return FLAGS_SET(t->answer_query_flags, SD_RESOLVED_AUTHENTICATED);
return FLAGS_SET(dt->answer_query_flags, SD_RESOLVED_AUTHENTICATED);
}

return true;
Expand Down

0 comments on commit 6da5ca9

Please sign in to comment.