diff options
author | David Woodhouse <David.Woodhouse@intel.com> | 2009-01-27 18:39:23 -0500 |
---|---|---|
committer | David Woodhouse <David.Woodhouse@intel.com> | 2009-01-27 18:39:23 -0500 |
commit | 3456b22111be920e15e6999b15d2f402a48e775d (patch) | |
tree | 97537437eea1b4d67eb303cde0856e398bb55bac /drivers/atm/solos-pci.c | |
parent | af7806560c972b5b8c79e9704d7816813343cbc1 (diff) |
solos: Fix under-allocation of skb size for get/set parameters
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/atm/solos-pci.c')
-rw-r--r-- | drivers/atm/solos-pci.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/atm/solos-pci.c b/drivers/atm/solos-pci.c index 297869965fc4..2dca5ffc8063 100644 --- a/drivers/atm/solos-pci.c +++ b/drivers/atm/solos-pci.c | |||
@@ -159,7 +159,7 @@ static ssize_t solos_param_show(struct device *dev, struct device_attribute *att | |||
159 | 159 | ||
160 | buflen = strlen(attr->attr.name) + 10; | 160 | buflen = strlen(attr->attr.name) + 10; |
161 | 161 | ||
162 | skb = alloc_skb(buflen, GFP_KERNEL); | 162 | skb = alloc_skb(sizeof(*header) + buflen, GFP_KERNEL); |
163 | if (!skb) { | 163 | if (!skb) { |
164 | dev_warn(&card->dev->dev, "Failed to allocate sk_buff in solos_param_show()\n"); | 164 | dev_warn(&card->dev->dev, "Failed to allocate sk_buff in solos_param_show()\n"); |
165 | return -ENOMEM; | 165 | return -ENOMEM; |
@@ -215,7 +215,7 @@ static ssize_t solos_param_store(struct device *dev, struct device_attribute *at | |||
215 | 215 | ||
216 | buflen = strlen(attr->attr.name) + 11 + count; | 216 | buflen = strlen(attr->attr.name) + 11 + count; |
217 | 217 | ||
218 | skb = alloc_skb(buflen, GFP_KERNEL); | 218 | skb = alloc_skb(sizeof(*header) + buflen, GFP_KERNEL); |
219 | if (!skb) { | 219 | if (!skb) { |
220 | dev_warn(&card->dev->dev, "Failed to allocate sk_buff in solos_param_store()\n"); | 220 | dev_warn(&card->dev->dev, "Failed to allocate sk_buff in solos_param_store()\n"); |
221 | return -ENOMEM; | 221 | return -ENOMEM; |