diff options
| author | Mathias Krause <minipli@googlemail.com> | 2013-11-05 21:25:32 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-11-07 14:10:41 +0100 |
| commit | 9968e459f1378a1c00301472cfe672497fe001d5 (patch) | |
| tree | eaf1f2cc63029b4f2abfb2a940f62726558926dc /drivers/acpi/event.c | |
| parent | ACPI: Fix spelling mistake in error messages (diff) | |
| download | linux-9968e459f1378a1c00301472cfe672497fe001d5.tar.xz linux-9968e459f1378a1c00301472cfe672497fe001d5.zip | |
ACPI / event: remove unneeded NULL pointer check
nla_data() cannot return NULL as it simply adds NLA_HDRLEN to the
passed pointer. The NULL pointer check is therefore superfluous.
Remove it.
Found by smatch:
drivers/acpi/event.c:130 acpi_bus_generate_netlink_event() warn: can
'event' even be NULL?
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/event.c')
| -rw-r--r-- | drivers/acpi/event.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/acpi/event.c b/drivers/acpi/event.c index 8247fcdde079..fdef416c0ff6 100644 --- a/drivers/acpi/event.c +++ b/drivers/acpi/event.c @@ -127,11 +127,6 @@ int acpi_bus_generate_netlink_event(const char *device_class, } event = nla_data(attr); - if (!event) { - nlmsg_free(skb); - return -EINVAL; - } - memset(event, 0, sizeof(struct acpi_genl_event)); strcpy(event->device_class, device_class); |