[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <486B8FCF.8080201@yahoo.it>
Date:	Wed, 02 Jul 2008 16:25:19 +0200
From:	Filippo Zangheri <filippo.zangheri@...oo.it>
To:	Darren Jenkins <darrenrjenkins@...il.com>
CC:	kernel Janitors <kernel-janitors@...r.kernel.org>,
	LKML <linux-kernel@...r.kernel.org>,
	ipw2100-admin@...ux.intel.com, yi.zhu@...el.com,
	reinette.chatre@...el.com, linux-wireless@...r.kernel.org
Subject: Re: [PATCH]  drivers/net/wireless/iwlwifi/iwl-3945.c
(Cc'ing linux-wireless)
On 02/07/2008 03:37, Darren Jenkins\ wrote:
> G'day people
> 
> Coverity CID: 2265 NEGATIVE_RETURNS
> 
> "rate" is of an unsigned type, and the code requires a signed type.
> The following patch makes it so.
> 
> Only compile tested.
> 
> Signed-off-by: Darren Jenkins <darrenrjenkins@...ilcom>
> 
> diff --git a/drivers/net/wireless/iwlwifi/iwl-3945.c b/drivers/net/wireless/iwlwifi/iwl-3945.c
> index 62a3d8f..31d38cd 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-3945.c
> +++ b/drivers/net/wireless/iwlwifi/iwl-3945.c
> @@ -449,7 +449,7 @@ static void iwl3945_dbg_report_frame(struct iwl3945_priv *priv,
>  
>  	if (print_summary) {
>  		char *title;
> -		u32 rate;
> +		int rate;
>  
>  		if (hundred)
>  			title = "100Frames";
-- 
Filippo Zangheri
GPG key ID: 0x6C1F2F2F
--
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/
Powered by blists - more mailing lists
 
