Projects
Eulaceura:Factory
avahi
_service:obs_scm:backport-CVE-2023-38469.patch
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:obs_scm:backport-CVE-2023-38469.patch of Package avahi
From a337a1ba7d15853fb56deef1f464529af6e3a1cf Mon Sep 17 00:00:00 2001 From: Evgeny Vereshchagin <evvers@ya.ru> Date: Mon, 23 Oct 2023 20:29:31 +0000 Subject: [PATCH 1/2] core: reject overly long TXT resource records Closes https://github.com/lathiat/avahi/issues/455 CVE-2023-38469 Reference:https://github.com/lathiat/avahi/commit/61b9874ff91dd20a12483db07df29fe7f35db77f Conflict:Adaptation Context --- avahi-core/rr.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/avahi-core/rr.c b/avahi-core/rr.c index 2bb89244..9c04ebbd 100644 --- a/avahi-core/rr.c +++ b/avahi-core/rr.c @@ -32,6 +32,7 @@ #include <avahi-common/malloc.h> #include <avahi-common/defs.h> +#include "dns.h" #include "rr.h" #include "log.h" #include "util.h" @@ -689,11 +690,17 @@ int avahi_record_is_valid(AvahiRecord *r) { case AVAHI_DNS_TYPE_TXT: { AvahiStringList *strlst; + size_t used = 0; - for (strlst = r->data.txt.string_list; strlst; strlst = strlst->next) + for (strlst = r->data.txt.string_list; strlst; strlst = strlst->next) { if (strlst->size > 255 || strlst->size <= 0) return 0; + used += 1+strlst->size; + if (used > AVAHI_DNS_RDATA_MAX) + return 0; + } + return 1; } } From c6cab87df290448a63323c8ca759baa516166237 Mon Sep 17 00:00:00 2001 From: Evgeny Vereshchagin <evvers@ya.ru> Date: Wed, 25 Oct 2023 18:15:42 +0000 Subject: [PATCH 2/2] tests: pass overly long TXT resource records to make sure they don't crash avahi any more. It reproduces https://github.com/lathiat/avahi/issues/455 --- avahi-client/client-test.c | 14 ++++++++++++++ 1 files changed, 14 insertions(+) diff --git a/avahi-client/client-test.c b/avahi-client/client-test.c index ba979988..da0e43ad 100644 --- a/avahi-client/client-test.c +++ b/avahi-client/client-test.c @@ -22,6 +22,7 @@ #endif #include <stdio.h> +#include <string.h> #include <assert.h> #include <avahi-client/client.h> @@ -33,6 +34,8 @@ #include <avahi-common/malloc.h> #include <avahi-common/timeval.h> +#include <avahi-core/dns.h> + static const AvahiPoll *poll_api = NULL; static AvahiSimplePoll *simple_poll = NULL; @@ -222,6 +225,9 @@ int main (AVAHI_GCC_UNUSED int argc, AVAHI_GCC_UNUSED char *argv[]) { uint32_t cookie; struct timeval tv; AvahiAddress a; + uint8_t rdata[AVAHI_DNS_RDATA_MAX+1]; + AvahiStringList *txt = NULL; + int r; simple_poll = avahi_simple_poll_new(); poll_api = avahi_simple_poll_get(simple_poll); @@ -261,6 +267,14 @@ int main (AVAHI_GCC_UNUSED int argc, AVAHI_GCC_UNUSED char *argv[]) { error = avahi_entry_group_add_record (group, AVAHI_IF_UNSPEC, AVAHI_PROTO_UNSPEC, 0, "TestX", 0x01, 0x10, 120, "", 0); assert(error != AVAHI_OK); + memset(rdata, 1, sizeof(rdata)); + r = avahi_string_list_parse(rdata, sizeof(rdata), &txt); + assert(r >= 0); + assert(avahi_string_list_serialize(txt, NULL, 0) == sizeof(rdata)); + error = avahi_entry_group_add_service_strlst(group, AVAHI_IF_UNSPEC, AVAHI_PROTO_UNSPEC, 0, "TestX", "_qotd._tcp", NULL, NULL, 123, txt); + assert(error == AVAHI_ERR_INVALID_RECORD); + avahi_string_list_free(txt); + avahi_entry_group_commit (group); domain = avahi_domain_browser_new (avahi, AVAHI_IF_UNSPEC, AVAHI_PROTO_UNSPEC, NULL, AVAHI_DOMAIN_BROWSER_BROWSE, 0, avahi_domain_browser_callback, (char*) "omghai3u");
Locations
Projects
Search
Status Monitor
Help
Open Build Service
OBS Manuals
API Documentation
OBS Portal
Reporting a Bug
Contact
Mailing List
Forums
Chat (IRC)
Twitter
Open Build Service (OBS)
is an
openSUSE project
.
浙ICP备2022010568号-2