summaryrefslogtreecommitdiffstats
path: root/fs/nfs
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2014-04-05 08:45:57 -0400
committerTrond Myklebust <trond.myklebust@primarydata.com>2014-04-05 09:25:42 -0400
commit9581a4ae75517099bc87e1c43d1a8f35b55741b9 (patch)
treebe0688b7e4af19715e11f82f83f86664a61e9116 /fs/nfs
parente911b8158ee1def8153849b1641b736026b036e0 (diff)
downloadop-kernel-dev-9581a4ae75517099bc87e1c43d1a8f35b55741b9.zip
op-kernel-dev-9581a4ae75517099bc87e1c43d1a8f35b55741b9.tar.gz
nfs: pass string length to pr_notice message about readdir loops
There is no guarantee that the strings in the nfs_cache_array will be NULL-terminated. In the event that we end up hitting a readdir loop, we need to ensure that we pass the warning message the length of the string. Reported-by: Lachlan McIlroy <lmcilroy@redhat.com> Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'fs/nfs')
-rw-r--r--fs/nfs/dir.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c
index b31f5d2..ef3fd09 100644
--- a/fs/nfs/dir.c
+++ b/fs/nfs/dir.c
@@ -313,10 +313,9 @@ int nfs_readdir_search_for_cookie(struct nfs_cache_array *array, nfs_readdir_des
if (printk_ratelimit()) {
pr_notice("NFS: directory %pD2 contains a readdir loop."
"Please contact your server vendor. "
- "The file: %s has duplicate cookie %llu\n",
- desc->file,
- array->array[i].string.name,
- *desc->dir_cookie);
+ "The file: %.*s has duplicate cookie %llu\n",
+ desc->file, array->array[i].string.len,
+ array->array[i].string.name, *desc->dir_cookie);
}
status = -ELOOP;
goto out;
OpenPOWER on IntegriCloud