gfs2: Fix fall-through warnings for Clang

In preparation to enable -Wimplicit-fallthrough for Clang, fix multiple
warnings by explicitly adding multiple goto statements instead of just
letting the code fall through to the next case.

Link: https://github.com/KSPP/linux/issues/115
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>

diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
index b44b487..f7f9ee7 100644 (file)
--- a/fs/gfs2/inode.c
+++ b/fs/gfs2/inode.c
@@ -963,6 +963,7 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir,
                break;
        case 0:
                error = -EEXIST;
+               goto out_gunlock;
        default:
                goto out_gunlock;
        }
@@ -1508,6 +1509,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
                        break;
                case 0:
                        error = -EEXIST;
+                       goto out_gunlock;
                default:
                        goto out_gunlock;
                }

diff --git a/fs/gfs2/recovery.c b/fs/gfs2/recovery.c
index 4f3fd35..016ed1b 100644 (file)
--- a/fs/gfs2/recovery.c
+++ b/fs/gfs2/recovery.c
@@ -438,6 +438,7 @@ void gfs2_recover_func(struct work_struct *work)
                case GLR_TRYFAILED:
                        fs_info(sdp, "jid=%u: Busy\n", jd->jd_jid);
                        error = 0;
+                       goto fail;
 
                default:
                        goto fail;