diff options
| author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2020-11-20 21:25:03 +0300 |
|---|---|---|
| committer | Andreas Gruenbacher <agruenba@redhat.com> | 2021-04-20 23:38:21 +0300 |
| commit | e5966cf20f0c7e40fd8c208ba1614e1a35a8deee (patch) | |
| tree | 9cecdef2d451791e17f19c867d6fa7d969c51931 /fs/gfs2 | |
| parent | c551f66c5dfefd00687f7567183fec6889ba46a3 (diff) | |
| download | linux-e5966cf20f0c7e40fd8c208ba1614e1a35a8deee.tar.xz | |
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>
Diffstat (limited to 'fs/gfs2')
| -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; |