The branch, master has been updated
       via  21cb6c8 torture: Use a short torture socket dir
       via  be46c97 swrap: Fix recvmsg() with UDP
      from  73b2b4f swrap: Fix compare of signed and unsigned integer 
expressions

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


- Log -----------------------------------------------------------------
commit 21cb6c86761c6a2db85a81d85395315d83f0cb52
Author: Andreas Schneider <a...@samba.org>
Date:   Tue Oct 27 10:03:06 2015 +0100

    torture: Use a short torture socket dir
    
    This makes sure we do not run into a regression.
    
    Thanks to Grigorij Demidov <grigorii.demi...@nic.cz>
    
    Signed-off-by: Andreas Schneider <a...@samba.org>
    Reviewed-by: Stefan Metzmacher <me...@samba.org>

commit be46c97142b9e79c45c8fdf078ac5245813738ba
Author: Andreas Schneider <a...@samba.org>
Date:   Tue Oct 27 09:26:40 2015 +0100

    swrap: Fix recvmsg() with UDP
    
    This worked if the unix path was bigger than sizeof(struct sockaddr_in6).
    With a short unix socket pathname the buffer was to small to store the
    address and convert_un_in() failed.
    
    Thanks to Grigorij Demidov <grigorii.demi...@nic.cz>
    
    Signed-off-by: Andreas Schneider <a...@samba.org>
    Reviewed-by: Stefan Metzmacher <me...@samba.org>

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

Summary of changes:
 src/socket_wrapper.c | 10 ++++++++++
 tests/torture.c      |  2 +-
 2 files changed, 11 insertions(+), 1 deletion(-)


Changeset truncated at 500 lines:

diff --git a/src/socket_wrapper.c b/src/socket_wrapper.c
index 6691738..3c0c279 100644
--- a/src/socket_wrapper.c
+++ b/src/socket_wrapper.c
@@ -4603,6 +4603,9 @@ static ssize_t swrap_recvmsg(int s, struct msghdr *omsg, 
int flags)
        struct swrap_address from_addr = {
                .sa_socklen = sizeof(struct sockaddr_un),
        };
+       struct swrap_address convert_addr = {
+               .sa_socklen = sizeof(struct sockaddr_storage),
+       };
        struct socket_info *si;
        struct msghdr msg;
        struct iovec tmp;
@@ -4661,6 +4664,13 @@ static ssize_t swrap_recvmsg(int s, struct msghdr *omsg, 
int flags)
        }
 #endif
 
+       /*
+        * We convert the unix address to a IP address so we need a buffer
+        * which can store the address in case of SOCK_DGRAM, see below.
+        */
+       msg.msg_name = &convert_addr.sa;
+       msg.msg_namelen = convert_addr.sa_socklen;
+
        rc = swrap_recvmsg_after(s,
                                 si,
                                 &msg,
diff --git a/tests/torture.c b/tests/torture.c
index 821f0ab..d3ad84a 100644
--- a/tests/torture.c
+++ b/tests/torture.c
@@ -55,7 +55,7 @@
 #define TORTURE_ECHO_SRV_IPV6 "fd00::5357:5f0a"
 #define TORTURE_ECHO_SRV_PORT 7
 
-#define TORTURE_SOCKET_DIR "/tmp/test_socket_wrapper_XXXXXX"
+#define TORTURE_SOCKET_DIR "/tmp/w_XXXXXX"
 #define TORTURE_ECHO_SRV_PIDFILE "echo_srv.pid"
 #define TORTURE_PCAP_FILE "socket_trace.pcap"
 


-- 
Socket Wrapper Repository

Reply via email to