aboutsummaryrefslogtreecommitdiffstats
path: root/main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch
diff options
context:
space:
mode:
Diffstat (limited to 'main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch')
-rw-r--r--main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch84
1 files changed, 0 insertions, 84 deletions
diff --git a/main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch b/main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch
deleted file mode 100644
index 160d4d0d6d7..00000000000
--- a/main/dnsmasq/0008-Fix-coverity-warnings-on-dbus.patch
+++ /dev/null
@@ -1,84 +0,0 @@
-Patch-Source: https://src.fedoraproject.org/rpms/dnsmasq/blob/f36/f/0008-Fix-coverity-warnings-on-dbus.patch (backport from upstream)
---
-From bbfdf6a435cbd5f71ae76f962ce86786346589aa Mon Sep 17 00:00:00 2001
-From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= <pemensik@redhat.com>
-Date: Fri, 3 Sep 2021 17:19:05 +0200
-Subject: [PATCH 08/15] Fix coverity warnings on dbus
-
-diff --git a/src/dbus.c b/src/dbus.c
-index cbdce9c..d746b9a 100644
---- a/src/dbus.c
-+++ b/src/dbus.c
-@@ -114,7 +114,7 @@ static dbus_bool_t add_watch(DBusWatch *watch, void *data)
- w->next = daemon->watches;
- daemon->watches = w;
-
-- w = data; /* no warning */
-+ (void)data; /* no warning */
- return TRUE;
- }
-
-@@ -134,16 +134,20 @@ static void remove_watch(DBusWatch *watch, void *data)
- up = &(w->next);
- }
-
-- w = data; /* no warning */
-+ (void)data; /* no warning */
- }
-
--static void dbus_read_servers(DBusMessage *message)
-+static DBusMessage* dbus_read_servers(DBusMessage *message)
- {
- DBusMessageIter iter;
- union mysockaddr addr, source_addr;
- char *domain;
-
-- dbus_message_iter_init(message, &iter);
-+ if (!dbus_message_iter_init(message, &iter))
-+ {
-+ return dbus_message_new_error(message, DBUS_ERROR_INVALID_ARGS,
-+ "Failed to initialize dbus message iter");
-+ }
-
- mark_servers(SERV_FROM_DBUS);
-
-@@ -222,6 +226,7 @@ static void dbus_read_servers(DBusMessage *message)
-
- /* unlink and free anything still marked. */
- cleanup_servers();
-+ return NULL;
- }
-
- #ifdef HAVE_LOOP
-@@ -545,6 +550,10 @@ static DBusMessage *dbus_add_lease(DBusMessage* message)
- "Invalid IP address '%s'", ipaddr);
-
- hw_len = parse_hex((char*)hwaddr, dhcp_chaddr, DHCP_CHADDR_MAX, NULL, &hw_type);
-+ if (hw_len < 0)
-+ return dbus_message_new_error_printf(message, DBUS_ERROR_INVALID_ARGS,
-+ "Invalid HW address '%s'", hwaddr);
-+
- if (hw_type == 0 && hw_len != 0)
- hw_type = ARPHRD_ETHER;
-
-@@ -668,7 +677,7 @@ DBusHandlerResult message_handler(DBusConnection *connection,
- #endif
- else if (strcmp(method, "SetServers") == 0)
- {
-- dbus_read_servers(message);
-+ reply = dbus_read_servers(message);
- new_servers = 1;
- }
- else if (strcmp(method, "SetServersEx") == 0)
-@@ -719,7 +728,7 @@ DBusHandlerResult message_handler(DBusConnection *connection,
- if (clear_cache)
- clear_cache_and_reload(dnsmasq_time());
-
-- method = user_data; /* no warning */
-+ (void)user_data; /* no warning */
-
- /* If no reply or no error, return nothing */
- if (!reply)
---
-2.31.1
-