diff options
| author | Zhao Hongjiang <zhaohongjiang@huawei.com> | 2013-06-20 12:59:51 +0200 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2013-06-25 19:53:06 +0200 |
| commit | ad917e7f821855a2f223131bb6c90ca6c9240bf3 (patch) | |
| tree | ab187928712bbf3a394658a3d6b393cb0553bc27 /fs/dlm | |
| parent | dlm: remove duplicated include from lowcomms.c (diff) | |
| download | linux-ad917e7f821855a2f223131bb6c90ca6c9240bf3.tar.xz linux-ad917e7f821855a2f223131bb6c90ca6c9240bf3.zip | |
dlm: config: using strlcpy instead of strncpy
for NUL terminated string, need alway set '\0' in the end.
Signed-off-by: Zhao Hongjiang <zhaohongjiang@huawei.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm')
| -rw-r--r-- | fs/dlm/config.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/dlm/config.c b/fs/dlm/config.c index 7d58d5b112b5..76feb4b60fa6 100644 --- a/fs/dlm/config.c +++ b/fs/dlm/config.c @@ -138,8 +138,9 @@ static ssize_t cluster_cluster_name_read(struct dlm_cluster *cl, char *buf) static ssize_t cluster_cluster_name_write(struct dlm_cluster *cl, const char *buf, size_t len) { - strncpy(dlm_config.ci_cluster_name, buf, DLM_LOCKSPACE_LEN); - strncpy(cl->cl_cluster_name, buf, DLM_LOCKSPACE_LEN); + strlcpy(dlm_config.ci_cluster_name, buf, + sizeof(dlm_config.ci_cluster_name)); + strlcpy(cl->cl_cluster_name, buf, sizeof(cl->cl_cluster_name)); return len; } |