diff options
author | Zhang Mingyu <zhang.mingyu@zte.com.cn> | 2021-11-12 12:25:47 +0300 |
---|---|---|
committer | Dominique Martinet <asmadeus@codewreck.org> | 2022-01-10 04:00:09 +0300 |
commit | a7a427d1543f89ad57fda5d6b9bb70f4cecb2fba (patch) | |
tree | d87ac4dd7d5d024735d706889ccf29c901bacaff /fs/9p/vfs_file.c | |
parent | 019641d1b57dff018972b23c95e898f9ff18222f (diff) | |
download | linux-a7a427d1543f89ad57fda5d6b9bb70f4cecb2fba.tar.xz |
9p: Use BUG_ON instead of if condition followed by BUG.
This issue was detected with the help of Coccinelle.
Link: https://lkml.kernel.org/r/20211112092547.9153-1-zhang.mingyu@zte.com.cn
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Zhang Mingyu <zhang.mingyu@zte.com.cn>
Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
Diffstat (limited to 'fs/9p/vfs_file.c')
-rw-r--r-- | fs/9p/vfs_file.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c index b42b65e15b81..ed0c85a2fb89 100644 --- a/fs/9p/vfs_file.c +++ b/fs/9p/vfs_file.c @@ -138,8 +138,7 @@ static int v9fs_file_do_lock(struct file *filp, int cmd, struct file_lock *fl) fid = filp->private_data; BUG_ON(fid == NULL); - if ((fl->fl_flags & FL_POSIX) != FL_POSIX) - BUG(); + BUG_ON((fl->fl_flags & FL_POSIX) != FL_POSIX); res = locks_lock_file_wait(filp, fl); if (res < 0) |