|
| 1 | +From a2696da2f2c50ac43b6c4903f72290d5c3fa9f6f Mon Sep 17 00:00:00 2001 |
| 2 | +From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= < [email protected]> |
| 3 | +Date: Thu, 17 Nov 2022 01:51:53 +0100 |
| 4 | +Subject: [PATCH] Emit error if requested service is not found |
| 5 | + |
| 6 | +It currently just crashes instead of replying with error. Check return |
| 7 | +value and emit error instead of passing NULL pointer to reply. |
| 8 | + |
| 9 | +Fixes #375 |
| 10 | +--- |
| 11 | + avahi-daemon/dbus-protocol.c | 20 ++++++++++++++------ |
| 12 | + 1 file changed, 14 insertions(+), 6 deletions(-) |
| 13 | + |
| 14 | +diff --git a/avahi-daemon/dbus-protocol.c b/avahi-daemon/dbus-protocol.c |
| 15 | +index 70d7687b..406d0b44 100644 |
| 16 | +--- a/avahi-daemon/dbus-protocol.c |
| 17 | ++++ b/avahi-daemon/dbus-protocol.c |
| 18 | +@@ -375,10 +375,14 @@ static DBusHandlerResult dbus_get_alternative_host_name(DBusConnection *c, DBusM |
| 19 | + } |
| 20 | + |
| 21 | + t = avahi_alternative_host_name(n); |
| 22 | +- avahi_dbus_respond_string(c, m, t); |
| 23 | +- avahi_free(t); |
| 24 | ++ if (t) { |
| 25 | ++ avahi_dbus_respond_string(c, m, t); |
| 26 | ++ avahi_free(t); |
| 27 | + |
| 28 | +- return DBUS_HANDLER_RESULT_HANDLED; |
| 29 | ++ return DBUS_HANDLER_RESULT_HANDLED; |
| 30 | ++ } else { |
| 31 | ++ return avahi_dbus_respond_error(c, m, AVAHI_ERR_NOT_FOUND, "Hostname not found"); |
| 32 | ++ } |
| 33 | + } |
| 34 | + |
| 35 | + static DBusHandlerResult dbus_get_alternative_service_name(DBusConnection *c, DBusMessage *m, DBusError *error) { |
| 36 | +@@ -389,10 +393,14 @@ static DBusHandlerResult dbus_get_alternative_service_name(DBusConnection *c, DB |
| 37 | + } |
| 38 | + |
| 39 | + t = avahi_alternative_service_name(n); |
| 40 | +- avahi_dbus_respond_string(c, m, t); |
| 41 | +- avahi_free(t); |
| 42 | ++ if (t) { |
| 43 | ++ avahi_dbus_respond_string(c, m, t); |
| 44 | ++ avahi_free(t); |
| 45 | + |
| 46 | +- return DBUS_HANDLER_RESULT_HANDLED; |
| 47 | ++ return DBUS_HANDLER_RESULT_HANDLED; |
| 48 | ++ } else { |
| 49 | ++ return avahi_dbus_respond_error(c, m, AVAHI_ERR_NOT_FOUND, "Service not found"); |
| 50 | ++ } |
| 51 | + } |
| 52 | + |
| 53 | + static DBusHandlerResult dbus_create_new_entry_group(DBusConnection *c, DBusMessage *m, DBusError *error) { |
0 commit comments