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/client/daemon-client.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/client/daemon-client.c')
-rw-r--r-- | libdaemon/client/daemon-client.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/libdaemon/client/daemon-client.c b/libdaemon/client/daemon-client.c index 136c57d1..906de38c 100644 --- a/libdaemon/client/daemon-client.c +++ b/libdaemon/client/daemon-client.c @@ -32,7 +32,10 @@ daemon_handle daemon_open(daemon_info i) { goto error; memset(&sockaddr, 0, sizeof(sockaddr)); - strncpy(sockaddr.sun_path, i.socket, sizeof(sockaddr.sun_path)); + if (!dm_strncpy(sockaddr.sun_path, i.socket, sizeof(sockaddr.sun_path))) { + fprintf(stderr, "%s: daemon socket path too long.\n", i.socket); + goto error; + } sockaddr.sun_family = AF_UNIX; if (connect(h.socket_fd,(struct sockaddr *) &sockaddr, sizeof(sockaddr))) goto error; |