diff options
author | Axel Lin <axel.lin@ingics.com> | 2012-10-27 11:03:00 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2012-11-17 05:17:28 -0500 |
commit | 64ebe955f9447fcecf65360ec3bc332f4ec411b6 (patch) | |
tree | 73ef17a2dd1c4f6958facd12b2516c55342f454a /drivers/iio/light | |
parent | 95273f8952a7e3a037e139eec0aa99872f086498 (diff) |
iio: hid-sensor: Return proper error if kmemdup fails
Return -ENOMEM instead of 0 if kmemdup fails.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio/light')
-rw-r--r-- | drivers/iio/light/hid-sensor-als.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c index 8e1f69844eea..23eeeef64e84 100644 --- a/drivers/iio/light/hid-sensor-als.c +++ b/drivers/iio/light/hid-sensor-als.c | |||
@@ -272,10 +272,9 @@ static int __devinit hid_als_probe(struct platform_device *pdev) | |||
272 | goto error_free_dev; | 272 | goto error_free_dev; |
273 | } | 273 | } |
274 | 274 | ||
275 | channels = kmemdup(als_channels, | 275 | channels = kmemdup(als_channels, sizeof(als_channels), GFP_KERNEL); |
276 | sizeof(als_channels), | ||
277 | GFP_KERNEL); | ||
278 | if (!channels) { | 276 | if (!channels) { |
277 | ret = -ENOMEM; | ||
279 | dev_err(&pdev->dev, "failed to duplicate channels\n"); | 278 | dev_err(&pdev->dev, "failed to duplicate channels\n"); |
280 | goto error_free_dev; | 279 | goto error_free_dev; |
281 | } | 280 | } |