lib: Remove use of MSG_NOSIGNAL
authorVolker Lendecke <vl@samba.org>
Mon, 29 May 2017 14:15:50 +0000 (16:15 +0200)
committerRalph Boehme <slow@samba.org>
Tue, 30 May 2017 14:39:12 +0000 (16:39 +0200)
According to susv4 sendmsg, NOSIGAL is effective for stream-oriented sockets.
Datagram sockets won't send SIGPIPE anyway. Looking at Linux kernel sources,
this is only looked at in stream functions. I guess this is a left-over from my
tmsgd attempts, which was based on stream sockets. messaging_dgm still only
uses datagram sockets, so MSG_NOSIGNAL is not needed here.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12502

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Tue May 30 16:39:12 CEST 2017 on sn-devel-144

source3/lib/messages_dgm.c

index dff8357f60d99b2b69d12a8cfa3f7864cc95698f..095a319f94a92f96ec263bbfeddc02d757638df5 100644 (file)
@@ -363,7 +363,7 @@ static ssize_t messaging_dgm_sendmsg(int sock,
                msghdr_prep_fds(&msg, buf, fdlen, fds, num_fds);
 
                do {
-                       ret = sendmsg(sock, &msg, MSG_NOSIGNAL);
+                       ret = sendmsg(sock, &msg, 0);
                } while ((ret == -1) && (errno == EINTR));
        }