diff options
| author | J. Bruce Fields <bfields@citi.umich.edu> | 2007-06-30 18:40:32 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@citi.umich.edu> | 2007-07-19 01:09:27 +0200 |
| commit | d2ab0b0c4c2570921a9ec1eff1e3a5143e05b231 (patch) | |
| tree | 3ca5586f765066acae9e2c20eb5578ce3220e4e8 | |
| parent | leases: minor break_lease() comment clarification (diff) | |
| download | linux-d2ab0b0c4c2570921a9ec1eff1e3a5143e05b231.tar.xz linux-d2ab0b0c4c2570921a9ec1eff1e3a5143e05b231.zip | |
locks: convert an -EINVAL return to a BUG
There's no point trying to return an error in these cases, which all represent
bugs in the callers.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
| -rw-r--r-- | fs/locks.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/locks.c b/fs/locks.c index 924e4a7fb1d5..3c23fd261022 100644 --- a/fs/locks.c +++ b/fs/locks.c @@ -1348,9 +1348,7 @@ static int __setlease(struct file *filp, long arg, struct file_lock **flp) time_out_leases(inode); - error = -EINVAL; - if (!flp || !(*flp) || !(*flp)->fl_lmops || !(*flp)->fl_lmops->fl_break) - goto out; + BUG_ON(!(*flp)->fl_lmops->fl_break); lease = *flp; |