Jeff Layton has uploaded this change for review.

View Change

NFS: properly handle NULL fs_locations pointer

RFC5661, section 11.9:

When the fs_locations attribute is interrogated and there are no
alternate file system locations, the server SHOULD return a zero-
length array of fs_location4 structures, together with a valid
fs_root.

If we have a NULL fs_locations pointer, we currently fill in a
structure with a bunch of "not supported" strings. Change the code
to handle it more in line with what the spec suggests.

There is one wrinkle here, which is that if we don't have an
fs_locations structure, then we also don't have an fs_root. In that
case, just assume that the entire export has gone away and set the
fs_root to the pseudopath of the export.

Change-Id: I11e4db806dcbf3c853b066b5a97f950c57ec66c1
Signed-off-by: Jeff Layton <jlayton@redhat.com>
---
M src/Protocols/NFS/nfs_proto_tools.c
1 file changed, 16 insertions(+), 18 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/99/422099/1

To view, visit change 422099. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I11e4db806dcbf3c853b066b5a97f950c57ec66c1
Gerrit-Change-Number: 422099
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton@redhat.com>