Projects
Mega:23.09
avahi
_service:tar_scm:0005-avahi_dns_packet_consume_...
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:tar_scm:0005-avahi_dns_packet_consume_uint32-fix-potential-undefi.patch of Package avahi
From b897ca43ac100d326d118e5877da710eb7f836f9 Mon Sep 17 00:00:00 2001 From: traffic-millions <60914101+traffic-millions@users.noreply.github.com> Date: Tue, 3 Mar 2020 11:15:48 +0800 Subject: [PATCH 11/11] avahi_dns_packet_consume_uint32: fix potential undefined behavior avahi_dns_packet_consume_uint32 left shifts uint8_t values by 8, 16 and 24 bits to combine them into a 32-bit value. This produces an undefined behavior warning with gcc -fsanitize when fed input values of 128 or 255 however in testing no actual unexpected behavior occurs in practice and the 32-bit uint32_t is always correctly produced as the final value is immediately stored into a uint32_t and the compiler appears to handle this "correctly". Cast the intermediate values to uint32_t to prevent this warning and ensure the intended result is explicit. Closes: #267 Closes: #268 Reference: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=19304 --- avahi-core/dns.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/avahi-core/dns.c b/avahi-core/dns.c index 7c38f42..d793b76 100644 --- a/avahi-core/dns.c +++ b/avahi-core/dns.c @@ -455,7 +455,7 @@ int avahi_dns_packet_consume_uint32(AvahiDnsPacket *p, uint32_t *ret_v) { return -1; d = (uint8_t*) (AVAHI_DNS_PACKET_DATA(p) + p->rindex); - *ret_v = (d[0] << 24) | (d[1] << 16) | (d[2] << 8) | d[3]; + *ret_v = ((uint32_t)d[0] << 24) | ((uint32_t)d[1] << 16) | ((uint32_t)d[2] << 8) | (uint32_t)d[3]; p->rindex += sizeof(uint32_t); return 0; -- 2.25.2
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