On 21/05/14 09:02, Dominique Martinet wrote: > v9fs_fid_xattr_set is supposed to return 0 on success. > > This corrects the behaviour introduced in commit > bdd5c28dcb8330b9074404cc92a0b83aae5606a > "9p: fix return value in case in v9fs_fid_xattr_set()" > > (The function returns a negative error on error, as expected) > > Signed-off-by: Dominique Martinet <dominique.martinet@xxxxxx> > --- > fs/9p/xattr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/9p/xattr.c b/fs/9p/xattr.c > index 04133a1..f95e01e 100644 > --- a/fs/9p/xattr.c > +++ b/fs/9p/xattr.c > @@ -156,7 +156,7 @@ int v9fs_fid_xattr_set(struct p9_fid *fid, const char *name, > offset += write_count; > value_len -= write_count; > } > - retval = offset; > + retval = 0; > err: > p9_client_clunk(fid); > return retval; > This patch seems to be fixing the problem, thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
