Commit 17590840 authored by Oliver Neukum's avatar Oliver Neukum Committed by Greg Kroah-Hartman

[PATCH] USB: kzalloc in PhidgetInterfaceKit

another for kzalloc.
Signed-off-by: default avatarOliver Neukum <oliver@neukum.name>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 1144cf7a
...@@ -88,7 +88,7 @@ static int change_outputs(struct phidget_interfacekit *kit, int output_num, int ...@@ -88,7 +88,7 @@ static int change_outputs(struct phidget_interfacekit *kit, int output_num, int
int retval; int retval;
int n; int n;
buffer = kmalloc(4, GFP_KERNEL); buffer = kzalloc(4, GFP_KERNEL);
if (!buffer) { if (!buffer) {
dev_err(&kit->udev->dev, "%s - out of memory\n", dev_err(&kit->udev->dev, "%s - out of memory\n",
__FUNCTION__); __FUNCTION__);
...@@ -96,7 +96,6 @@ static int change_outputs(struct phidget_interfacekit *kit, int output_num, int ...@@ -96,7 +96,6 @@ static int change_outputs(struct phidget_interfacekit *kit, int output_num, int
} }
kit->outputs[output_num] = enable; kit->outputs[output_num] = enable;
memset(buffer, 0, 4);
for (n=0; n<8; n++) { for (n=0; n<8; n++) {
if (kit->outputs[n]) { if (kit->outputs[n]) {
buffer[0] |= 1 << n; buffer[0] |= 1 << n;
...@@ -192,7 +191,7 @@ static ssize_t set_backlight(struct device *dev, struct device_attribute *attr, ...@@ -192,7 +191,7 @@ static ssize_t set_backlight(struct device *dev, struct device_attribute *attr,
unsigned char *buffer; unsigned char *buffer;
int retval = -ENOMEM; int retval = -ENOMEM;
buffer = kmalloc(8, GFP_KERNEL); buffer = kzalloc(8, GFP_KERNEL);
if (!buffer) { if (!buffer) {
dev_err(&kit->udev->dev, "%s - out of memory\n", __FUNCTION__); dev_err(&kit->udev->dev, "%s - out of memory\n", __FUNCTION__);
goto exit; goto exit;
...@@ -202,7 +201,6 @@ static ssize_t set_backlight(struct device *dev, struct device_attribute *attr, ...@@ -202,7 +201,6 @@ static ssize_t set_backlight(struct device *dev, struct device_attribute *attr,
retval = -EINVAL; retval = -EINVAL;
goto exit; goto exit;
} }
memset(buffer, 0x00, 8);
if (enabled) if (enabled)
buffer[0] = 0x01; buffer[0] = 0x01;
buffer[7] = 0x11; buffer[7] = 0x11;
...@@ -406,12 +404,11 @@ static int interfacekit_probe(struct usb_interface *intf, const struct usb_devic ...@@ -406,12 +404,11 @@ static int interfacekit_probe(struct usb_interface *intf, const struct usb_devic
pipe = usb_rcvintpipe(dev, endpoint->bEndpointAddress); pipe = usb_rcvintpipe(dev, endpoint->bEndpointAddress);
maxp = usb_maxpacket(dev, pipe, usb_pipeout(pipe)); maxp = usb_maxpacket(dev, pipe, usb_pipeout(pipe));
kit = kmalloc(sizeof(*kit), GFP_KERNEL); kit = kzalloc(sizeof(*kit), GFP_KERNEL);
if (kit == NULL) { if (kit == NULL) {
dev_err(&intf->dev, "%s - out of memory\n", __FUNCTION__); dev_err(&intf->dev, "%s - out of memory\n", __FUNCTION__);
return -ENOMEM; return -ENOMEM;
} }
memset(kit, 0, sizeof(*kit));
kit->ifkit = ifkit; kit->ifkit = ifkit;
kit->data = usb_buffer_alloc(dev, 8, SLAB_ATOMIC, &kit->data_dma); kit->data = usb_buffer_alloc(dev, 8, SLAB_ATOMIC, &kit->data_dma);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment