123456789101112131415161718192021222324252627282930313233343536373839404142434445464748 |
- Upstream-Status: Backport [https://github.com/lathiat/avahi/commit/e111def]
- CVE: CVE-2017-6519
- Signed-off-by: Kai Kang <kai.kang@windriver.com>
- From e111def44a7df4624a4aa3f85fe98054bffb6b4f Mon Sep 17 00:00:00 2001
- From: Trent Lloyd <trent@lloyd.id.au>
- Date: Sat, 22 Dec 2018 09:06:07 +0800
- Subject: [PATCH] Drop legacy unicast queries from address not on local link
- When handling legacy unicast queries, ensure that the source IP is
- inside a subnet on the local link, otherwise drop the packet.
- Fixes #145
- Fixes #203
- CVE-2017-6519
- CVE-2018-1000845
- ---
- avahi-core/server.c | 8 ++++++++
- 1 file changed, 8 insertions(+)
- diff --git a/avahi-core/server.c b/avahi-core/server.c
- index a2cb19a8..a2580e38 100644
- --- a/avahi-core/server.c
- +++ b/avahi-core/server.c
- @@ -930,6 +930,7 @@ static void dispatch_packet(AvahiServer *s, AvahiDnsPacket *p, const AvahiAddres
-
- if (avahi_dns_packet_is_query(p)) {
- int legacy_unicast = 0;
- + char t[AVAHI_ADDRESS_STR_MAX];
-
- /* For queries EDNS0 might allow ARCOUNT != 0. We ignore the
- * AR section completely here, so far. Until the day we add
- @@ -947,6 +948,13 @@ static void dispatch_packet(AvahiServer *s, AvahiDnsPacket *p, const AvahiAddres
- legacy_unicast = 1;
- }
-
- + if (!is_mdns_mcast_address(dst_address) &&
- + !avahi_interface_address_on_link(i, src_address)) {
- +
- + avahi_log_debug("Received non-local unicast query from host %s on interface '%s.%i'.", avahi_address_snprint(t, sizeof(t), src_address), i->hardware->name, i->protocol);
- + return;
- + }
- +
- if (legacy_unicast)
- reflect_legacy_unicast_query_packet(s, p, i, src_address, port);
-
|