diff options
| author |   Dan Carpenter <error27@gmail.com> | 2010-07-10 10:53:38 +0200 |
|---|---|---|
| committer |   John W. Linville <linville@tuxdriver.com> | 2010-07-12 16:05:32 -0400 |
| commit | 4c92831d99827971cf2854cfa7f884213099428f (patch) | |
| tree | fafce6b1a9c082952a91659d18e7f7d5970dd45c /drivers/net | |
| parent | wl1271: use __packed annotation (diff) | |
| download | linux-dev-4c92831d99827971cf2854cfa7f884213099428f.tar.xz linux-dev-4c92831d99827971cf2854cfa7f884213099428f.zip | |
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>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/wireless/prism54/isl_ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/prism54/isl_ioctl.c b/drivers/net/wireless/prism54/isl_ioctl.c index 236e37526d07..2e656bb7e14e 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); |