locks: convert an -EINVAL return to a BUG
authorJ. Bruce Fields <bfields@citi.umich.edu>
Sat, 30 Jun 2007 16:40:32 +0000 (12:40 -0400)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Wed, 18 Jul 2007 23:09:27 +0000 (19:09 -0400)
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>
fs/locks.c

index 924e4a7..3c23fd2 100644 (file)
@@ -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;