[SCM] Samba Shared Repository - branch master updated

Ralph Böhme slow at samba.org
Tue May 30 14:40:02 UTC 2017


The branch, master has been updated
       via  ccd4f81 lib: Remove use of MSG_NOSIGNAL
      from  41981db s4:lib/com: remove unused pycom binding

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit ccd4f814882029eba6aac2ce379a82420dc5f79e
Author: Volker Lendecke <vl at samba.org>
Date:   Mon May 29 16:15:50 2017 +0200

    lib: Remove use of MSG_NOSIGNAL
    
    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 at samba.org>
    Reviewed-by: Ralph Boehme <slow at samba.org>
    
    Autobuild-User(master): Ralph Böhme <slow at samba.org>
    Autobuild-Date(master): Tue May 30 16:39:12 CEST 2017 on sn-devel-144

-----------------------------------------------------------------------

Summary of changes:
 source3/lib/messages_dgm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/messages_dgm.c b/source3/lib/messages_dgm.c
index dff8357..095a319 100644
--- a/source3/lib/messages_dgm.c
+++ b/source3/lib/messages_dgm.c
@@ -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));
 	}
 


-- 
Samba Shared Repository



More information about the samba-cvs mailing list