lkml.org 
[lkml]   [2012]   [Sep]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] sysctl: Use BUG_ON instead of BUG
On Mon,  3 Sep 2012 09:00:25 +0530
Prasad Joshi <prasadjoshi.linux@gmail.com> wrote:

> The use of if (!head) BUG(); can be replaced with single line
> BUG_ON(!head).
>
> Signed-off-by: Prasad Joshi <prasadjoshi.linux@gmail.com>
> ---
> fs/proc/proc_sysctl.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
> index dfafeb2..63bdfa0 100644
> --- a/fs/proc/proc_sysctl.c
> +++ b/fs/proc/proc_sysctl.c
> @@ -266,8 +266,7 @@ void sysctl_head_put(struct ctl_table_header *head)
>
> static struct ctl_table_header *sysctl_head_grab(struct ctl_table_header *head)
> {
> - if (!head)
> - BUG();
> + BUG_ON(!head);
> spin_lock(&sysctl_lock);
> if (!use_table(head))
> head = ERR_PTR(-ENOENT);

We could just remove the check altogether. The resulting NULL pointer
deref oops in use_table() will give us the same info as that BUG().



\
 
 \ /
  Last update: 2012-09-05 03:42    [W:0.042 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site