[<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