Looks good to me. Thanks. Reviewed-by: Joseph Qi <[email protected]>
On 2014/11/6 15:07, Jan Kara wrote: > Error handling if creation of root of debugfs in ocfs2_init() fails is > broken. Although error code is set we fail to exit ocfs2_init() with > error and thus initialization ends with success. Later when mounting a > filesystem, ocfs2 debugfs entries end up being created in the root of > debugfs filesystem which is confusing. > > Fix the error handling to bail out. > > Coverity-id: 1227009 > Signed-off-by: Jan Kara <[email protected]> > --- > fs/ocfs2/super.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > This replaces patch "ocfs2: Remove pointless assignment in ocfs2_init()". > Things are safer this way and when debugfs dir creation fails, something is > really wrong. > > diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c > index 93c85bc745e1..8e3ac25efb9f 100644 > --- a/fs/ocfs2/super.c > +++ b/fs/ocfs2/super.c > @@ -1622,8 +1622,9 @@ static int __init ocfs2_init(void) > > ocfs2_debugfs_root = debugfs_create_dir("ocfs2", NULL); > if (!ocfs2_debugfs_root) { > - status = -EFAULT; > + status = -ENOMEM; > mlog(ML_ERROR, "Unable to create ocfs2 debugfs root.\n"); > + goto out4; > } > > ocfs2_set_locking_protocol(); > _______________________________________________ Ocfs2-devel mailing list [email protected] https://oss.oracle.com/mailman/listinfo/ocfs2-devel
