From c3cbf988b9e5ce27c82463bc85e468b667fe9de3 Mon Sep 17 00:00:00 2001

From: "Bernat, Yehezkel" <yehezkel.bernat@intel.com>

Date: Tue, 15 Aug 2017 08:19:01 +0300

Subject: [PATCH 31/34] thunderbolt: Remove superfluous check

Git-commit: 8fdd6ab36197ad891233572c57781b1f537da0ac

Patch-mainline: v4.14

References: FATE#323948

The key size is tested by hex2bin() already (as '\0' isn't an hex digit)

Suggested-by: Andy Shevchenko <andriy.shevchenko@intel.com>

Signed-off-by: Yehezkel Bernat <yehezkel.bernat@intel.com>

Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Signed-off-by: Oliver Neukum <oneukum@suse.com>

---

 drivers/thunderbolt/switch.c | 3 ---

 1 file changed, 3 deletions(-)

diff --git a/drivers/thunderbolt/switch.c b/drivers/thunderbolt/switch.c

index e9391bbd4036..69fde0b30eda 100644

--- a/drivers/thunderbolt/switch.c

+++ b/drivers/thunderbolt/switch.c

@@ -808,9 +808,6 @@ static ssize_t key_store(struct device *dev, struct device_attribute *attr,

 	u8 key[TB_SWITCH_KEY_SIZE];

 	ssize_t ret = count;

-	if (count < 64)

-		return -EINVAL;

-

 	if (hex2bin(key, buf, sizeof(key)))

 		return -EINVAL;

-- 

2.12.3