diff options
author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2020-11-20 19:25:03 +0100 |
---|---|---|
committer | Andreas Gruenbacher <agruenba@redhat.com> | 2021-04-20 22:38:21 +0200 |
commit | e5966cf20f0c7e40fd8c208ba1614e1a35a8deee (patch) | |
tree | 9cecdef2d451791e17f19c867d6fa7d969c51931 | |
parent | gfs2: Fix a number of kernel-doc warnings (diff) | |
download | linux-e5966cf20f0c7e40fd8c208ba1614e1a35a8deee.tar.xz linux-e5966cf20f0c7e40fd8c208ba1614e1a35a8deee.zip |
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>
-rw-r--r-- | fs/gfs2/inode.c | 2 | ||||
-rw-r--r-- | fs/gfs2/recovery.c | 1 |
2 files changed, 3 insertions, 0 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index b44b4873b35f..f7f9ee723617 100644 --- 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 4f3fd351c140..016ed1b2ca1d 100644 --- 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; |