diff options
| author | David Sterba <dsterba@suse.cz> | 2013-11-19 07:36:21 -0500 |
|---|---|---|
| committer | Chris Mason <clm@fb.com> | 2014-01-28 16:19:41 -0500 |
| commit | cc37bb04201217b7acb11213e16cb5530c30da8f (patch) | |
| tree | c27e7ecb38db667734de53619ebb00995d41521d /fs/btrfs/sysfs.c | |
| parent | 43d87fa23154d135a2a1006bc6656ae73ae84190 (diff) | |
btrfs: replace BUG in can_modify_feature
We don't need to crash hard here, it's just reading a sysfs file. The
values considered in switch are from a fixed set, the default case
should not happen at all.
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Josef Bacik <jbacik@fb.com>
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs/btrfs/sysfs.c')
| -rw-r--r-- | fs/btrfs/sysfs.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sysfs.c index 8fdc0520e298..b535285642dc 100644 --- a/fs/btrfs/sysfs.c +++ b/fs/btrfs/sysfs.c | |||
| @@ -75,7 +75,9 @@ static int can_modify_feature(struct btrfs_feature_attr *fa) | |||
| 75 | clear = BTRFS_FEATURE_INCOMPAT_SAFE_CLEAR; | 75 | clear = BTRFS_FEATURE_INCOMPAT_SAFE_CLEAR; |
| 76 | break; | 76 | break; |
| 77 | default: | 77 | default: |
| 78 | BUG(); | 78 | printk(KERN_WARNING "btrfs: sysfs: unknown feature set %d\n", |
| 79 | fa->feature_set); | ||
| 80 | return 0; | ||
| 79 | } | 81 | } |
| 80 | 82 | ||
| 81 | if (set & fa->feature_bit) | 83 | if (set & fa->feature_bit) |
