From a4445e88cece0444c66d70876b03065158dd4685 Mon Sep 17 00:00:00 2001 From: Tom Gundersen Date: Wed, 11 Mar 2015 22:23:38 +0100 Subject: [PATCH] libudev: monitor - fix error path in send_device Return -errno rather than -1 in case sendmsg() fails. --- src/libudev/libudev-monitor.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/libudev/libudev-monitor.c b/src/libudev/libudev-monitor.c index eb7b6f87b..0c554bbac 100644 --- a/src/libudev/libudev-monitor.c +++ b/src/libudev/libudev-monitor.c @@ -745,12 +745,20 @@ int udev_monitor_send_device(struct udev_monitor *udev_monitor, * If we send to a multicast group, we will get * ECONNREFUSED, which is expected. */ - if (destination != NULL) + if (destination) smsg.msg_name = &destination->snl; else smsg.msg_name = &udev_monitor->snl_destination; smsg.msg_namelen = sizeof(struct sockaddr_nl); count = sendmsg(udev_monitor->sock, &smsg, 0); + if (count < 0) { + if (!destination && errno == ECONNREFUSED) { + log_debug("passed unknown number of bytes to netlink monitor %p", udev_monitor); + return 0; + } else + return -errno; + } + log_debug("passed %zi bytes to netlink monitor %p", count, udev_monitor); return count; } -- 2.30.2