netdev - [PATCH] ARM: fix bug of checking on signed return value using unsigned statement in w90p910 platform

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:	Wed, 11 Nov 2009 22:35:22 +0800
From:	Wan ZongShun <mcuos.com@...il.com>
To:	"David S. Miller" <davem@...emloft.net>, roel.kluin@...il.com
CC:	linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
	linux-netdev <netdev@...r.kernel.org>
Subject: [PATCH] ARM: fix bug of checking on signed return value using unsigned
 statement in w90p910 platform

Dear sir,

To fix the bug of checking on signed return value using unsigned statement.
Thanks Roel Kluin for digging out it. 

Signed-off-by: Roel Kluin <roel.kluin@...il.com>
Signed-off-by: Wan ZongShun <mcuos.com@...il.com>

---
 drivers/net/arm/w90p910_ether.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/arm/w90p910_ether.c b/drivers/net/arm/w90p910_ether.c
index 25e2627..b7f3866 100644
--- a/drivers/net/arm/w90p910_ether.c
+++ b/drivers/net/arm/w90p910_ether.c
@@ -160,8 +160,8 @@ struct  w90p910_ether {
 	struct mii_if_info mii;
 	struct timer_list check_timer;
 	void __iomem *reg;
-	unsigned int rxirq;
-	unsigned int txirq;
+	int rxirq;
+	int txirq;
 	unsigned int cur_tx;
 	unsigned int cur_rx;
 	unsigned int finish_tx;
-- 
1.5.6.3
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html