Clarify uxsock logging
authorHannes Reinecke <hare@suse.de>
Tue, 26 Nov 2013 11:41:22 +0000 (12:41 +0100)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Sat, 30 Nov 2013 14:45:32 +0000 (15:45 +0100)
Socket creation might fail on various stages, so print out a
proper logging message.

Signed-off-by: Hannes Reinecke <hare@suse.de>
libmultipath/uxsock.c
multipathd/uxclnt.c
multipathd/uxlsnr.c

index fcad56e..e93edff 100644 (file)
@@ -19,6 +19,7 @@
 
 #include "memory.h"
 #include "uxsock.h"
+#include "debug.h"
 
 /*
  * connect to a unix domain socket
@@ -36,10 +37,12 @@ int ux_socket_connect(const char *name)
 
        fd = socket(AF_LOCAL, SOCK_STREAM, 0);
        if (fd == -1) {
+               condlog(3, "Couldn't create ux_socket, error %d", errno);
                return -1;
        }
 
        if (connect(fd, (struct sockaddr *)&addr, len) == -1) {
+               condlog(3, "Couldn't connect to ux_socket, error %d", errno);
                close(fd);
                return -1;
        }
@@ -57,7 +60,10 @@ int ux_socket_listen(const char *name)
        struct sockaddr_un addr;
 
        fd = socket(AF_LOCAL, SOCK_STREAM, 0);
-       if (fd == -1) return -1;
+       if (fd == -1) {
+               condlog(3, "Couldn't create ux_socket, error %d", errno);
+               return -1;
+       }
 
        memset(&addr, 0, sizeof(addr));
        addr.sun_family = AF_LOCAL;
@@ -66,15 +72,16 @@ int ux_socket_listen(const char *name)
        strncpy(&addr.sun_path[1], name, len);
 
        if (bind(fd, (struct sockaddr *)&addr, len) == -1) {
+               condlog(3, "Couldn't bind to ux_socket, error %d", errno);
                close(fd);
                return -1;
        }
 
        if (listen(fd, 10) == -1) {
+               condlog(3, "Couldn't listen to ux_socket, error %d", errno);
                close(fd);
                return -1;
        }
-
        return fd;
 }
 
index 3e4e381..e86be21 100644 (file)
@@ -121,10 +121,8 @@ int uxclnt(char * inbuf)
        int fd;
 
        fd = ux_socket_connect(DEFAULT_SOCKET);
-       if (fd == -1) {
-               perror("ux_socket_connect");
+       if (fd == -1)
                exit(1);
-       }
 
        if (inbuf)
                process_req(fd, inbuf);
index c0ddd8d..ed8e012 100644 (file)
@@ -112,10 +112,8 @@ void * uxsock_listen(int (*uxsock_trigger)(char *, char **, int *, void *),
 
        ux_sock = ux_socket_listen(DEFAULT_SOCKET);
 
-       if (ux_sock == -1) {
-               condlog(0, "ux_socket_listen error");
+       if (ux_sock == -1)
                exit(1);
-       }
 
        pthread_cleanup_push(uxsock_cleanup, NULL);