linux-kernel - Re: drivers/net/phy/marvell.c:1037:5-8: WARNING: Unsigned expression compared with zero: val

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>] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.10.1602170835480.2685@hadrien>
Date:	Wed, 17 Feb 2016 08:36:49 -0500 (EST)
From:	Julia Lawall <julia.lawall@...6.fr>
To:	Andrew Lunn <andrew@...n.ch>
cc:	kbuild-all@...org, linux-kernel@...r.kernel.org,
	Florian Fainelli <f.fainelli@...il.com>
Subject: Re: drivers/net/phy/marvell.c:1037:5-8: WARNING: Unsigned expression
 compared with zero: val < 0

Looks like a problem, please check.

julia


> >> drivers/net/phy/marvell.c:1037:5-8: WARNING: Unsigned expression compared with zero: val < 0
>
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git remote update linus
> git checkout d2fa47d9dd5c5f82d78c4503cf33989f67e8aa64
> vim +1037 drivers/net/phy/marvell.c
>
> d2fa47d9 Andrew Lunn 2015-12-30  1021  #define UINT64_MAX              (u64)(~((u64)0))
> d2fa47d9 Andrew Lunn 2015-12-30  1022  #endif
> d2fa47d9 Andrew Lunn 2015-12-30  1023  static u64 marvell_get_stat(struct phy_device *phydev, int i)
> d2fa47d9 Andrew Lunn 2015-12-30  1024  {
> d2fa47d9 Andrew Lunn 2015-12-30  1025  	struct marvell_hw_stat stat = marvell_hw_stats[i];
> d2fa47d9 Andrew Lunn 2015-12-30  1026  	struct marvell_priv *priv = phydev->priv;
> d2fa47d9 Andrew Lunn 2015-12-30  1027  	int err, oldpage;
> d2fa47d9 Andrew Lunn 2015-12-30  1028  	u64 val;
> d2fa47d9 Andrew Lunn 2015-12-30  1029
> d2fa47d9 Andrew Lunn 2015-12-30  1030  	oldpage = phy_read(phydev, MII_MARVELL_PHY_PAGE);
> d2fa47d9 Andrew Lunn 2015-12-30  1031  	err = phy_write(phydev, MII_MARVELL_PHY_PAGE,
> d2fa47d9 Andrew Lunn 2015-12-30  1032  			stat.page);
> d2fa47d9 Andrew Lunn 2015-12-30  1033  	if (err < 0)
> d2fa47d9 Andrew Lunn 2015-12-30  1034  		return UINT64_MAX;
> d2fa47d9 Andrew Lunn 2015-12-30  1035
> d2fa47d9 Andrew Lunn 2015-12-30  1036  	val = phy_read(phydev, stat.reg);
> d2fa47d9 Andrew Lunn 2015-12-30 @1037  	if (val < 0) {
> d2fa47d9 Andrew Lunn 2015-12-30  1038  		val = UINT64_MAX;
> d2fa47d9 Andrew Lunn 2015-12-30  1039  	} else {
> d2fa47d9 Andrew Lunn 2015-12-30  1040  		val = val & ((1 << stat.bits) - 1);
> d2fa47d9 Andrew Lunn 2015-12-30  1041  		priv->stats[i] += val;
> d2fa47d9 Andrew Lunn 2015-12-30  1042  		val = priv->stats[i];
> d2fa47d9 Andrew Lunn 2015-12-30  1043  	}
> d2fa47d9 Andrew Lunn 2015-12-30  1044
> d2fa47d9 Andrew Lunn 2015-12-30  1045  	phy_write(phydev, MII_MARVELL_PHY_PAGE, oldpage);
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
>