diff options
author | Peter Rajnoha <prajnoha@redhat.com> | 2012-04-27 09:52:33 +0000 |
---|---|---|
committer | Peter Rajnoha <prajnoha@redhat.com> | 2012-04-27 09:52:33 +0000 |
commit | bf29fa49d34fb72356ad53274b01bcc8481726dc (patch) | |
tree | 11b3bb34a4f3c152b211c50bd78a8f7207accebd /libdaemon/server/daemon-server.c | |
parent | e5b9338ada7152672000016861ef06f4af2160aa (diff) | |
download | lvm2-bf29fa49d34fb72356ad53274b01bcc8481726dc.tar.gz lvm2-bf29fa49d34fb72356ad53274b01bcc8481726dc.tar.xz lvm2-bf29fa49d34fb72356ad53274b01bcc8481726dc.zip |
Use dm_strncpy instead of plain strncpy while setting the socket path.
Also check the return value as it meaningless to work with truncated strings.
Diffstat (limited to 'libdaemon/server/daemon-server.c')
-rw-r--r-- | libdaemon/server/daemon-server.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/libdaemon/server/daemon-server.c b/libdaemon/server/daemon-server.c index ca5a30f3..568ef361 100644 --- a/libdaemon/server/daemon-server.c +++ b/libdaemon/server/daemon-server.c @@ -224,7 +224,10 @@ static int _open_socket(daemon_state s) fprintf(stderr, "[D] creating %s\n", s.socket_path); memset(&sockaddr, 0, sizeof(sockaddr)); - strncpy(sockaddr.sun_path, s.socket_path, sizeof(sockaddr.sun_path)); + if (!dm_strncpy(sockaddr.sun_path, s.socket_path, sizeof(sockaddr.sun_path))) { + fprintf(stderr, "%s: daemon socket path too long.\n", s.socket_path); + goto error; + } sockaddr.sun_family = AF_UNIX; if (bind(fd, (struct sockaddr *) &sockaddr, sizeof(sockaddr))) { |