multipathd: switch to abstract sockets for CLI commands
authorHannes Reinecke <hare@suse.de>
Thu, 17 Jan 2013 14:59:29 +0000 (15:59 +0100)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Fri, 18 Jan 2013 19:05:40 +0000 (20:05 +0100)
Instead of using file-based sockets we should be switching to
abstract sockets. Otherwise multipathd won't be able to start
if the directory is not available.

Signed-off-by: Hannes Reinecke <hare@suse.de>
libmultipath/defaults.h
libmultipath/uxsock.c

index 32948d8..b83d9fb 100644 (file)
@@ -24,7 +24,7 @@
 
 #define MAX_DEV_LOSS_TMO       0x7FFFFFFF
 #define DEFAULT_PIDFILE                "/var/run/multipathd.pid"
 
 #define MAX_DEV_LOSS_TMO       0x7FFFFFFF
 #define DEFAULT_PIDFILE                "/var/run/multipathd.pid"
-#define DEFAULT_SOCKET         "/var/run/multipathd.sock"
+#define DEFAULT_SOCKET         "/org/kernel/linux/storage/multipathd"
 #define DEFAULT_CONFIGFILE     "/etc/multipath.conf"
 #define DEFAULT_BINDINGS_FILE  "/etc/multipath/bindings"
 #define DEFAULT_WWIDS_FILE     "/etc/multipath/wwids"
 #define DEFAULT_CONFIGFILE     "/etc/multipath.conf"
 #define DEFAULT_BINDINGS_FILE  "/etc/multipath/bindings"
 #define DEFAULT_WWIDS_FILE     "/etc/multipath/wwids"
index e786899..f3e8dec 100644 (file)
  */
 int ux_socket_connect(const char *name)
 {
  */
 int ux_socket_connect(const char *name)
 {
-       int fd;
+       int fd, len;
        struct sockaddr_un addr;
 
        memset(&addr, 0, sizeof(addr));
        struct sockaddr_un addr;
 
        memset(&addr, 0, sizeof(addr));
-       addr.sun_family = AF_UNIX;
-       strncpy(addr.sun_path, name, sizeof(addr.sun_path));
+       addr.sun_family = AF_LOCAL;
+       addr.sun_path[0] = '\0';
+       len = strlen(name) + 1;
+       strncpy(&addr.sun_path[1], name, len);
 
 
-       fd = socket(AF_UNIX, SOCK_STREAM, 0);
+       fd = socket(AF_LOCAL, SOCK_STREAM, 0);
        if (fd == -1) {
                return -1;
        }
 
        if (fd == -1) {
                return -1;
        }
 
-       if (connect(fd, (struct sockaddr *)&addr, sizeof(addr)) == -1) {
+       if (connect(fd, (struct sockaddr *)&addr, len) == -1) {
                close(fd);
                return -1;
        }
                close(fd);
                return -1;
        }
@@ -51,20 +53,19 @@ int ux_socket_connect(const char *name)
  */
 int ux_socket_listen(const char *name)
 {
  */
 int ux_socket_listen(const char *name)
 {
-       int fd;
+       int fd, len;
        struct sockaddr_un addr;
 
        struct sockaddr_un addr;
 
-       /* get rid of any old socket */
-       unlink(name);
-
-       fd = socket(AF_UNIX, SOCK_STREAM, 0);
+       fd = socket(AF_LOCAL, SOCK_STREAM, 0);
        if (fd == -1) return -1;
 
        memset(&addr, 0, sizeof(addr));
        if (fd == -1) return -1;
 
        memset(&addr, 0, sizeof(addr));
-       addr.sun_family = AF_UNIX;
-       strncpy(addr.sun_path, name, sizeof(addr.sun_path));
+       addr.sun_family = AF_LOCAL;
+       addr.sun_path[0] = '\0';
+       len = strlen(name) + 1;
+       strncpy(&addr.sun_path[1], name, len);
 
 
-       if (bind(fd, (struct sockaddr *)&addr, sizeof(addr)) == -1) {
+       if (bind(fd, (struct sockaddr *)&addr, len) == -1) {
                close(fd);
                return -1;
        }
                close(fd);
                return -1;
        }