Commit e5966cf2 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Andreas Gruenbacher

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/115Signed-off-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: default avatarAndreas Gruenbacher <agruenba@redhat.com>
parent c551f66c
...@@ -963,6 +963,7 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir, ...@@ -963,6 +963,7 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir,
break; break;
case 0: case 0:
error = -EEXIST; error = -EEXIST;
goto out_gunlock;
default: default:
goto out_gunlock; goto out_gunlock;
} }
...@@ -1508,6 +1509,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry, ...@@ -1508,6 +1509,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
break; break;
case 0: case 0:
error = -EEXIST; error = -EEXIST;
goto out_gunlock;
default: default:
goto out_gunlock; goto out_gunlock;
} }
......
...@@ -438,6 +438,7 @@ void gfs2_recover_func(struct work_struct *work) ...@@ -438,6 +438,7 @@ void gfs2_recover_func(struct work_struct *work)
case GLR_TRYFAILED: case GLR_TRYFAILED:
fs_info(sdp, "jid=%u: Busy\n", jd->jd_jid); fs_info(sdp, "jid=%u: Busy\n", jd->jd_jid);
error = 0; error = 0;
goto fail;
default: default:
goto fail; goto fail;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment