From 42b63656b378c94522cc1ee53c658d22b8276465 Mon Sep 17 00:00:00 2001
From: Ben Hutchings <bhutchings@solarflare.com>
Date: Tue, 6 Sep 2011 12:44:25 +0000
Subject: [PATCH] --- yaml --- r: 264059 b: refs/heads/master c:
 710778ff878a06654175863db133293007d45aee h: refs/heads/master i:   264057:
 5b0ffc1ee432ebc1f725a2a4e9ec2ed038de4ec3   264055:
 e7dba6f823e7ea1a67ca2c056f950c91bc2b99f8 v: v3

---
 [refs]                              | 2 +-
 trunk/drivers/net/gianfar_ethtool.c | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/[refs] b/[refs]
index d923a7d186f5..3876530ca065 100644
--- a/[refs]
+++ b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 22e83a2926998fe132ae4dd26f1e998c70ae2e38
+refs/heads/master: 710778ff878a06654175863db133293007d45aee
diff --git a/trunk/drivers/net/gianfar_ethtool.c b/trunk/drivers/net/gianfar_ethtool.c
index 25a8c2adb001..0caf3c323ec0 100644
--- a/trunk/drivers/net/gianfar_ethtool.c
+++ b/trunk/drivers/net/gianfar_ethtool.c
@@ -1669,10 +1669,10 @@ static int gfar_get_cls_all(struct gfar_private *priv,
 	u32 i = 0;
 
 	list_for_each_entry(comp, &priv->rx_list.list, list) {
-		if (i <= cmd->rule_cnt) {
-			rule_locs[i] = comp->fs.location;
-			i++;
-		}
+		if (i == cmd->rule_cnt)
+			return -EMSGSIZE;
+		rule_locs[i] = comp->fs.location;
+		i++;
 	}
 
 	cmd->data = MAX_FILER_IDX;