Re: [PATCH -next] mm/usercopy: fix warning Comparison to bool

From: Andrew Morton
Date: Wed Apr 15 2020 - 19:04:55 EST


On Tue, 14 Apr 2020 11:42:04 +0800 Zou Wei <zou_wei@xxxxxxxxxx> wrote:

> fix below warnings reported by coccicheck
>
> mm/usercopy.c:304:5-18: WARNING: Comparison to bool
>
> ...
>
> --- a/mm/usercopy.c
> +++ b/mm/usercopy.c
> @@ -301,7 +301,7 @@ __setup("hardened_usercopy=", parse_hardened_usercopy);
>
> static int __init set_hardened_usercopy(void)
> {
> - if (enable_checks == false)
> + if (!enable_checks)
> static_branch_enable(&bypass_usercopy_checks);
> return 1;
> }

My initial reaction is "fix coccicheck". There's nothing wrong with
that code?