From: Dan Carpenter <error27@gmail.com>
Date: Sat, 10 Jul 2010 08:53:38 +0000 (+0200)
Subject: prism54: call BUG_ON() earlier
X-Git-Tag: 2.1b_release~7593^2~107^2~70
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4c92831d99827971cf2854cfa7f884213099428f;p=platform%2Fkernel%2Fkernel-mfld-blackbay.git

prism54: call BUG_ON() earlier

This test is off by one because strlen() doesn't include the NULL
terminator.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
---

diff --git a/drivers/net/wireless/prism54/isl_ioctl.c b/drivers/net/wireless/prism54/isl_ioctl.c
index 236e375..2e656bb 100644
--- a/drivers/net/wireless/prism54/isl_ioctl.c
+++ b/drivers/net/wireless/prism54/isl_ioctl.c
@@ -2067,7 +2067,7 @@ send_simple_event(islpci_private *priv, const char *str)
 	memptr = kmalloc(IW_CUSTOM_MAX, GFP_KERNEL);
 	if (!memptr)
 		return;
-	BUG_ON(n > IW_CUSTOM_MAX);
+	BUG_ON(n >= IW_CUSTOM_MAX);
 	wrqu.data.pointer = memptr;
 	wrqu.data.length = n;
 	strcpy(memptr, str);