diff options
author | Nicolas de Pesloüan <nicolas.2p.debian@free.fr> | 2009-08-28 09:18:34 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-08-29 02:01:16 -0400 |
commit | f584130616dfae757b888b7ee472e7c824f59e6a (patch) | |
tree | efe1382a091dfb6af18206472bf5472c32192e6a /drivers/net/bonding/bond_main.c | |
parent | 89c76c62f191daa7ede3d1d0c510a5ccfbcae571 (diff) |
bonding: Fix useless test: int > INT_MAX
max_bonds is of type int and cannot be greater than INT_MAX.
Signed-off-by: Nicolas de Pesloüan <nicolas.2p.debian@free.fr>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_main.c')
-rw-r--r-- | drivers/net/bonding/bond_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index ed00ba919d91..6b9f15b32a38 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c | |||
@@ -4759,7 +4759,7 @@ static int bond_check_params(struct bond_params *params) | |||
4759 | params->ad_select = BOND_AD_STABLE; | 4759 | params->ad_select = BOND_AD_STABLE; |
4760 | } | 4760 | } |
4761 | 4761 | ||
4762 | if (max_bonds < 0 || max_bonds > INT_MAX) { | 4762 | if (max_bonds < 0) { |
4763 | pr_warning(DRV_NAME | 4763 | pr_warning(DRV_NAME |
4764 | ": Warning: max_bonds (%d) not in range %d-%d, so it " | 4764 | ": Warning: max_bonds (%d) not in range %d-%d, so it " |
4765 | "was reset to BOND_DEFAULT_MAX_BONDS (%d)\n", | 4765 | "was reset to BOND_DEFAULT_MAX_BONDS (%d)\n", |