* Re: [PATCH] drivers/net/wireless/iwlwifi/iwl-3945.c
[not found] <1214962647.18261.15.camel@localhost.localdomain>
@ 2008-07-02 14:25 ` Filippo Zangheri
0 siblings, 0 replies; only message in thread
From: Filippo Zangheri @ 2008-07-02 14:25 UTC (permalink / raw)
To: Darren Jenkins
Cc: kernel Janitors, LKML, ipw2100-admin, yi.zhu, reinette.chatre,
linux-wireless
(Cc'ing linux-wireless)
On 02/07/2008 03:37, Darren Jenkins\ wrote:
> G'day people
>=20
> Coverity CID: 2265 NEGATIVE_RETURNS
>=20
> "rate" is of an unsigned type, and the code requires a signed type.
> The following patch makes it so.
>=20
> Only compile tested.
>=20
> =EF=BB=BFSigned-off-by: Darren Jenkins <darrenrjenkins@gmailcom>
>=20
> diff --git a/drivers/net/wireless/iwlwifi/iwl-3945.c b/drivers/net/wi=
reless/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 iwl39=
45_priv *priv,
> =20
> if (print_summary) {
> char *title;
> - u32 rate;
> + int rate;
> =20
> if (hundred)
> title =3D "100Frames";
--=20
=46ilippo Zangheri
GPG key ID: 0x6C1F2F2F
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] only message in thread