Re: [PATCH 3/3] ceph: fix dentry reference leak in ceph_encode_fh().

2012-11-20 Thread Alex Elder
On 11/11/2012 02:49 PM, Cyril Roelandt wrote: > dput() was not called in the error path. > > Signed-off-by: Cyril Roelandt This looks good, thanks a lot. I'll apply it. Reviewed-by: Alex Elder > --- > fs/ceph/export.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff

Re: [PATCH 3/3] ceph: fix dentry reference leak in ceph_encode_fh().

2012-11-20 Thread Alex Elder
On 11/11/2012 02:49 PM, Cyril Roelandt wrote: dput() was not called in the error path. Signed-off-by: Cyril Roelandt tipec...@gmail.com This looks good, thanks a lot. I'll apply it. Reviewed-by: Alex Elder el...@inktank.com --- fs/ceph/export.c |4 +++- 1 file changed, 3

[PATCH 3/3] ceph: fix dentry reference leak in ceph_encode_fh().

2012-11-11 Thread Cyril Roelandt
dput() was not called in the error path. Signed-off-by: Cyril Roelandt --- fs/ceph/export.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/fs/ceph/export.c b/fs/ceph/export.c index 9349bb3..ca3ab3f 100644 --- a/fs/ceph/export.c +++ b/fs/ceph/export.c @@ -56,13 +56,15

[PATCH 3/3] ceph: fix dentry reference leak in ceph_encode_fh().

2012-11-11 Thread Cyril Roelandt
dput() was not called in the error path. Signed-off-by: Cyril Roelandt tipec...@gmail.com --- fs/ceph/export.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/fs/ceph/export.c b/fs/ceph/export.c index 9349bb3..ca3ab3f 100644 --- a/fs/ceph/export.c +++ b/fs/ceph/export.c