linux-kernel - Re: [PATCH] staging: rts5208: remove always true comparison

Open Source and information security mailing list archives

Message-ID: <20150514151859.GW14154@mwanda>
Date:	Thu, 14 May 2015 18:18:59 +0300
From:	Dan Carpenter <dan.carpenter@...cle.com>
To:	Sudip Mukherjee <sudipm.mukherjee@...il.com>,
	Micky Ching <micky_ching@...lsil.com.cn>
Cc:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: rts5208: remove always true comparison

On Thu, May 14, 2015 at 07:55:59PM +0530, Sudip Mukherjee wrote:
> the if comparison is always true as anything bitwise and-ing with 0x1E
> can never be equal to 0x03.
> 
> Fixes bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=69881
> 

It doesn't fix anything though, it just silences the warning.  Warnings
are nice because they let you know that the code is buggy.  It's better
to leave them there instead of making the bugs more complicated to find.

Micky Ching might know if this is the right thing to do or if something
else was intended instead of 0x03?

regards,
dan carpenter

> Reported-by: David Binderman <dcb314@...mail.com>
> Signed-off-by: Sudip Mukherjee <sudip@...torindia.org>
> ---
>  drivers/staging/rts5208/sd.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/rts5208/sd.c b/drivers/staging/rts5208/sd.c
> index a8d657b..2e3f1f4 100644
> --- a/drivers/staging/rts5208/sd.c
> +++ b/drivers/staging/rts5208/sd.c
> @@ -4170,10 +4170,8 @@ RTY_SEND_CMD:
>  				}
>  
>  			} else if (rsp_type == SD_RSP_TYPE_R0) {
> -				if ((ptr[3] & 0x1E) != 0x03) {
> -					rtsx_trace(chip);
> -					return STATUS_FAIL;
> -				}
> +				rtsx_trace(chip);
> +				return STATUS_FAIL;
>  			}
>  		}
>  	}
> -- 
> 1.8.1.2
> 
> _______________________________________________
> devel mailing list
> devel@...uxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives