Commit c6399c16 authored by Neil Brown's avatar Neil Brown Committed by Linus Torvalds

[PATCH] locks: remove unnecessary BUG()

We get a NULL dereference here anyway, no need to BUG() explicitly.
Signed-off-by: default avatarJ. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: default avatarNeil Brown <neilb@cse.unsw.edu.au>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 77340644
...@@ -1172,10 +1172,8 @@ int __break_lease(struct inode *inode, unsigned int mode) ...@@ -1172,10 +1172,8 @@ int __break_lease(struct inode *inode, unsigned int mode)
if (fl->fl_type != future) { if (fl->fl_type != future) {
fl->fl_type = future; fl->fl_type = future;
fl->fl_break_time = break_time; fl->fl_break_time = break_time;
if (fl->fl_lmops && fl->fl_lmops->fl_break) /* lease must have lmops break callback */
fl->fl_lmops->fl_break(fl); fl->fl_lmops->fl_break(fl);
else /* lease must have lmops break callback */
BUG();
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment