diff options
author | Ondrej Zajicek <santiago@crfreenet.org> | 2008-10-26 23:55:38 +0100 |
---|---|---|
committer | Ondrej Zajicek <santiago@crfreenet.org> | 2008-10-26 23:55:38 +0100 |
commit | 68fa95cfec78f1bfe790949bf747d578ad583ec2 (patch) | |
tree | d46589a057106fcc9d249705dbcf909395e8bb49 | |
parent | 52586ccdf17cd9f34f767718d97f8a258da5efdb (diff) |
Check of socket name length
-rw-r--r-- | client/client.c | 4 | ||||
-rw-r--r-- | sysdep/unix/io.c | 4 |
2 files changed, 8 insertions, 0 deletions
diff --git a/client/client.c b/client/client.c index d65ea066..e3f539e1 100644 --- a/client/client.c +++ b/client/client.c @@ -252,6 +252,10 @@ server_connect(void) server_fd = socket(AF_UNIX, SOCK_STREAM, 0); if (server_fd < 0) die("Cannot create socket: %m"); + + if (strlen(server_path) >= sizeof(sa.sun_path)) + die("server_connect: path too long"); + bzero(&sa, sizeof(sa)); sa.sun_family = AF_UNIX; strcpy(sa.sun_path, server_path); diff --git a/sysdep/unix/io.c b/sysdep/unix/io.c index 7dcca21a..fa471f6e 100644 --- a/sysdep/unix/io.c +++ b/sysdep/unix/io.c @@ -865,6 +865,10 @@ sk_open_unix(sock *s, char *name) if (err = sk_setup(s)) goto bad; unlink(name); + + if (strlen(name) >= sizeof(sa.sun_path)) + die("sk_open_unix: path too long"); + sa.sun_family = AF_UNIX; strcpy(sa.sun_path, name); if (bind(fd, (struct sockaddr *) &sa, SUN_LEN(&sa)) < 0) |