Commit 02a2976c authored by Chuck Lever's avatar Chuck Lever Committed by Trond Myklebust

NFS: Clean up debugging in decode_pathname()

I noticed recently that decode_attr_fs_locations() is not generating
very pretty debugging output.  The pathname components each appear on
a separate line of output, though that does not appear to be the
intended display behavior.  The preferred way to generate continued
lines of output on the console is to use pr_cont().

Note that incoming pathname4 components contain a string that is not
necessarily NUL-terminated.  I did actually see some trailing garbage
on the console.  In addition to correcting the line continuation
problem, add a string precision format specifier to ensure that each
component string is displayed properly, and that vsnprintf() does
not Oops.

Someone pointed out that allowing incoming network data to possibly
generate a console line of unbounded length may not be such a good
idea.  Since this output will rarely be enabled, and there is a hard
upper bound (NFS4_PATHNAME_MAXCOMPONENTS) in our implementation, this
is probably not a major concern.

It might be useful to additionally sanity-check the length of each
incoming component, however.  RFC 3530bis15 does not suggest a maximum
number of UTF-8 characters per component for either the pathname4 or
component4 types.  However, we could invent one that is appropriate
for our implementation.

Another possibility is to scrap all of this and print these pathnames
in upper layers after a reasonable amount of sanity checking in the
XDR layer.  This would give us an opportunity to allocate a full
buffer so that the whole pathname would be output via a single
dprintk.

Introduced by commit 7aaa0b3b: "NFSv4: convert fs-locations-components
to conform to RFC3530," (June 9, 2006).
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 88b8e133
...@@ -3555,16 +3555,17 @@ static int decode_pathname(struct xdr_stream *xdr, struct nfs4_pathname *path) ...@@ -3555,16 +3555,17 @@ static int decode_pathname(struct xdr_stream *xdr, struct nfs4_pathname *path)
n = be32_to_cpup(p); n = be32_to_cpup(p);
if (n == 0) if (n == 0)
goto root_path; goto root_path;
dprintk("path "); dprintk("pathname4: ");
path->ncomponents = 0; path->ncomponents = 0;
while (path->ncomponents < n) { while (path->ncomponents < n) {
struct nfs4_string *component = &path->components[path->ncomponents]; struct nfs4_string *component = &path->components[path->ncomponents];
status = decode_opaque_inline(xdr, &component->len, &component->data); status = decode_opaque_inline(xdr, &component->len, &component->data);
if (unlikely(status != 0)) if (unlikely(status != 0))
goto out_eio; goto out_eio;
if (path->ncomponents != n) if (unlikely(nfs_debug & NFSDBG_XDR))
dprintk("/"); pr_cont("%s%.*s ",
dprintk("%s", component->data); (path->ncomponents != n ? "/ " : ""),
component->len, component->data);
if (path->ncomponents < NFS4_PATHNAME_MAXCOMPONENTS) if (path->ncomponents < NFS4_PATHNAME_MAXCOMPONENTS)
path->ncomponents++; path->ncomponents++;
else { else {
...@@ -3573,14 +3574,13 @@ static int decode_pathname(struct xdr_stream *xdr, struct nfs4_pathname *path) ...@@ -3573,14 +3574,13 @@ static int decode_pathname(struct xdr_stream *xdr, struct nfs4_pathname *path)
} }
} }
out: out:
dprintk("\n");
return status; return status;
root_path: root_path:
/* a root pathname is sent as a zero component4 */ /* a root pathname is sent as a zero component4 */
path->ncomponents = 1; path->ncomponents = 1;
path->components[0].len=0; path->components[0].len=0;
path->components[0].data=NULL; path->components[0].data=NULL;
dprintk("path /\n"); dprintk("pathname4: /\n");
goto out; goto out;
out_eio: out_eio:
dprintk(" status %d", status); dprintk(" status %d", status);
...@@ -3606,7 +3606,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st ...@@ -3606,7 +3606,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st
/* Ignore borken servers that return unrequested attrs */ /* Ignore borken servers that return unrequested attrs */
if (unlikely(res == NULL)) if (unlikely(res == NULL))
goto out; goto out;
dprintk("%s: fsroot ", __func__); dprintk("%s: fsroot:\n", __func__);
status = decode_pathname(xdr, &res->fs_path); status = decode_pathname(xdr, &res->fs_path);
if (unlikely(status != 0)) if (unlikely(status != 0))
goto out; goto out;
...@@ -3627,7 +3627,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st ...@@ -3627,7 +3627,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st
m = be32_to_cpup(p); m = be32_to_cpup(p);
loc->nservers = 0; loc->nservers = 0;
dprintk("%s: servers ", __func__); dprintk("%s: servers:\n", __func__);
while (loc->nservers < m) { while (loc->nservers < m) {
struct nfs4_string *server = &loc->servers[loc->nservers]; struct nfs4_string *server = &loc->servers[loc->nservers];
status = decode_opaque_inline(xdr, &server->len, &server->data); status = decode_opaque_inline(xdr, &server->len, &server->data);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment