aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@citi.umich.edu>2007-06-30 12:40:32 -0400
committerJ. Bruce Fields <bfields@citi.umich.edu>2007-07-18 19:09:27 -0400
commitd2ab0b0c4c2570921a9ec1eff1e3a5143e05b231 (patch)
tree3ca5586f765066acae9e2c20eb5578ce3220e4e8
parent87250dd26a34c65ae31d08837174222889007641 (diff)
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.c4
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)
1348 1348
1349 time_out_leases(inode); 1349 time_out_leases(inode);
1350 1350
1351 error = -EINVAL; 1351 BUG_ON(!(*flp)->fl_lmops->fl_break);
1352 if (!flp || !(*flp) || !(*flp)->fl_lmops || !(*flp)->fl_lmops->fl_break)
1353 goto out;
1354 1352
1355 lease = *flp; 1353 lease = *flp;
1356 1354