Hi Alexander,

2015-08-08 21:05 GMT+02:00 Alexander Kabaev <kab...@gmail.com>:
> this commit broke more than just syslogd. rtsol and rtsold both are
> victims of the change as they are trying to shutdown unconnected
> raw sockets to make then 'send-only' and fail.

Good catch! I think that in this case it is safe to omit the call to
shutdown(), or at least not let the application fail on startup
because of shutdown() being more strict. After all, this code would
fail on Linux, OS X, etc. already because of this.

Could you test the attached patch and let me know whether it makes
rtsol/rtsold work again?

Thanks,
-- 
Ed Schouten <e...@nuxi.nl>
Nuxi, 's-Hertogenbosch, the Netherlands
KvK/VAT number: 62051717
diff --git a/usr.sbin/rtsold/probe.c b/usr.sbin/rtsold/probe.c
index 89a1617a..3aa5303 100644
--- a/usr.sbin/rtsold/probe.c
+++ b/usr.sbin/rtsold/probe.c
@@ -80,12 +80,6 @@ probe_init(void)
                return (-1);
        }
 
-       /* make the socket send-only */
-       if (shutdown(probesock, 0)) {
-               warnmsg(LOG_ERR, __func__, "shutdown: %s", strerror(errno));
-               return (-1);
-       }
-
        /* initialize msghdr for sending packets */
        sndmhdr.msg_namelen = sizeof(struct sockaddr_in6);
        sndmhdr.msg_iov = sndiov;
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to