9p: fix readlink
authorMartin Stava <martin.stava@gmail.com>
Mon, 2 Nov 2009 14:39:34 +0000 (08:39 -0600)
committerEric Van Hensbergen <ericvh@gmail.com>
Mon, 2 Nov 2009 14:43:45 +0000 (08:43 -0600)
commit2511cd0b3b9e9b1c3e9360cc565c3745ac3f3f3f
tree70fff7f686a9c6c956146e39f656412715d40996
parentf91b90993f0d286be89f06c2f547ced8cfe291c6
9p: fix readlink

I do not know if you've looked on the patch, but unfortunately it is
incorrect. A suggested better version is in this email (the old
version didn't work in case the user provided buffer was not long
enough - it incorrectly appended null byte on a position of last char,
and thus broke the contract of the readlink method). However, I'm
still not sure this is 100% correct thing to do, I think readlink is
supposed to return buffer without last null byte in all cases, but we
do return last null byte (even the old version).. on the other hand it
is likely unspecified what is in the remaining part of the buffer, so
null character may be fine there ;):

Signed-off-by: Martin Stava <martin.stava@gmail.com>
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
fs/9p/vfs_inode.c