[Cluster-devel] [PATCH] gfs2: Improve gfs2_make_fs_rw error handling

[Andreas Gruenbacher]

In gfs2_make_fs_rw(), make sure to call gfs2_consist() to report an
inconsistency and mark the filesystem as withdrawn when
gfs2_find_jhead() fails.

At the end of gfs2_make_fs_rw(), when we discover that the filesystem
has been withdrawn, make sure we report an error.  This also replaces
the gfs2_withdrawn() check after gfs2_find_jhead().

Reported-by: Tetsuo Handa <penguin-ker...@i-love.sakura.ne.jp>
Cc: syzbot+f51cb4b9afbd87ec0...@syzkaller.appspotmail.com
Signed-off-by: Andreas Gruenbacher <agrue...@redhat.com>
---
 fs/gfs2/super.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
index 52c3502de58c..a83fa62106f0 100644
--- a/fs/gfs2/super.c
+++ b/fs/gfs2/super.c
@@ -138,8 +138,10 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
                return -EIO;
 
        error = gfs2_find_jhead(sdp->sd_jdesc, &head, false);
-       if (error || gfs2_withdrawn(sdp))
+       if (error) {
+               gfs2_consist(sdp);
                return error;
+       }
 
        if (!(head.lh_flags & GFS2_LOG_HEAD_UNMOUNT)) {
                gfs2_consist(sdp);
@@ -151,7 +153,9 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
        gfs2_log_pointers_init(sdp, head.lh_blkno);
 
        error = gfs2_quota_init(sdp);
-       if (!error && !gfs2_withdrawn(sdp))
+       if (!error && gfs2_withdrawn(sdp))
+               error = -EIO;
+       if (!error)
                set_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
        return error;
 }
-- 
2.39.0