diff options
author | Mario Limonciello <mario.limonciello@dell.com> | 2017-11-09 12:49:09 -0500 |
---|---|---|
committer | Darren Hart (VMware) <dvhart@infradead.org> | 2017-11-16 20:45:26 -0500 |
commit | 8b9528a6d9a901b9f933231505fef5630e80ce5a (patch) | |
tree | 3e6d5a9ea96b58b7e90f1459ef0dd7f6cc2e14c6 | |
parent | 73ed298b0605a81d8cbf212df887f7b2fe65a61b (diff) |
platform/x86: dell-wmi-descriptor: check if memory was allocated
devm_kzalloc will return NULL pointer if no memory was allocated.
This should be checked. This problem also existed when the driver
was dell-wmi.c.
Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
Reviewed-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
-rw-r--r-- | drivers/platform/x86/dell-wmi-descriptor.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/platform/x86/dell-wmi-descriptor.c b/drivers/platform/x86/dell-wmi-descriptor.c index 3204c408e261..28ef5f37cfbf 100644 --- a/drivers/platform/x86/dell-wmi-descriptor.c +++ b/drivers/platform/x86/dell-wmi-descriptor.c | |||
@@ -121,6 +121,11 @@ static int dell_wmi_descriptor_probe(struct wmi_device *wdev) | |||
121 | priv = devm_kzalloc(&wdev->dev, sizeof(struct descriptor_priv), | 121 | priv = devm_kzalloc(&wdev->dev, sizeof(struct descriptor_priv), |
122 | GFP_KERNEL); | 122 | GFP_KERNEL); |
123 | 123 | ||
124 | if (!priv) { | ||
125 | ret = -ENOMEM; | ||
126 | goto out; | ||
127 | } | ||
128 | |||
124 | priv->interface_version = buffer[2]; | 129 | priv->interface_version = buffer[2]; |
125 | priv->size = buffer[3]; | 130 | priv->size = buffer[3]; |
126 | ret = 0; | 131 | ret = 0; |