]> git.baikalelectronics.ru Git - kernel.git/commit
ACPI / watchdog: properly initialize resources
authorArnd Bergmann <arnd@arndb.de>
Mon, 18 Sep 2017 23:49:02 +0000 (01:49 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 18 Sep 2017 23:49:02 +0000 (01:49 +0200)
commit76b708a01d2fd253894268a4d661506b72b4f4da
tree53c1bf49a789953b69d5612d39f64e4715f993cf
parent2018862bdad13479b663e0e9615d7c6fcaa89285
ACPI / watchdog: properly initialize resources

We copy a local resource structure into a list, but only
initialize some of its members, as pointed out by gcc-4.4:

drivers/acpi/acpi_watchdog.c: In function 'acpi_watchdog_init':
drivers/acpi/acpi_watchdog.c:105: error: 'res.child' may be used uninitialized in this function
drivers/acpi/acpi_watchdog.c:105: error: 'res.sibling' may be used uninitialized in this function
drivers/acpi/acpi_watchdog.c:105: error: 'res.parent' may be used uninitialized in this function
drivers/acpi/acpi_watchdog.c:105: error: 'res.desc' may be used uninitialized in this function
drivers/acpi/acpi_watchdog.c:105: error: 'res.name' may be used uninitialized in this function

Newer compilers can presumably optimize the uninitialized access
away entirely and don't warn at all, but rely on the kzalloc()
to zero the structure first. This adds an explicit initialization
to force consistent behavior.

Fixes: 6395bf05f3e2 (ACPI / watchdog: Add support for WDAT hardware watchdog)
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/acpi_watchdog.c