0ct0pu5/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-12-04 05:20:30 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

LookupServer: Only pass A record responses to clients

Browse source 
This works around an issue where we'd try responding to a LookupServer
client with an empty CNAME record.
This commit is contained in:
Andreas Kling 2020-05-03 14:18:53 +02:00
parent 82d83f0fe1
commit 3c0ca04ad3
Notes: sideshowbarker 2024-07-19 07:01:15 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/3c0ca04ad31
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Servers/LookupServer/LookupServer.cpp
View file

@ -243,6 +243,8 @@ Vector<String> LookupServer::lookup(const String& hostname, bool& did_timeout, u
Vector<String, 8> responses;
Vector<DNSAnswer, 8> cacheable_answers;
for (auto& answer : response.answers()) {
if (answer.type() != T_A)
continue;
responses.append(answer.record_data());
if (!answer.has_expired())
cacheable_answers.append(answer);