From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751868AbdIOT4X (ORCPT ); Fri, 15 Sep 2017 15:56:23 -0400 Received: from mout.kundenserver.de ([212.227.126.135]:50613 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751825AbdIOT4V (ORCPT ); Fri, 15 Sep 2017 15:56:21 -0400 From: Arnd Bergmann To: "Rafael J. Wysocki" , Len Brown Cc: Arnd Bergmann , Mika Westerberg , Ryan Kennedy , Guenter Roeck , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] acpi: watchdog: properly initialize resources Date: Fri, 15 Sep 2017 21:55:18 +0200 Message-Id: <20170915195526.1541309-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:xH0xSJf7ItO1oMMJW0FtIi1ssWUPWRgr0Mz8DQCyh5mX+LDdKty GuEZ1ShUxKmr8d2/3kFzY2W8FUyM9DrZ3OyMUi6oNH/uWi/ytP9qH/SHyHKD9WnVcONbqgW 6F1e/tduzb0PSNLlYm3yBwA2NMpYPqUroly17X4xNbuKLQhGzGLl3Xw74wg2dNVAcZD+PH1 0V0mA3zI5oWSNg5RDaHMg== X-UI-Out-Filterresults: notjunk:1;V01:K0:aDkm8k6y6vU=:Md0vFdS+rdJLQwq/IWtLTp Yq+kvVq8E+t7yAmPR/DkGoUrlDG37FldLEqc+wz/kefsU3NEhUY8wBVjj3aWEbdD6lFZXuCz9 NP97PRd9tHaBNZozpEl+UAiOfm8uasrSJ3jUZQJY7PzmlpyykDuryelJgWGp4N3lqVJZXH8Sl IXKGNd1S9igzEsMHDDVIYhoj0Kl3QEMaJ8ByZgXaxUprjpvcrwRE058VtzSB4jAplIgqD8F4z U9XZvBDNBSMKIevGg2FJ3DN/1lsABNvYePgLbheSytmc47ViF/Dlmr2aaT3oMdyleTi0ji3E/ jDmYQkswY7rW8QVyPk1HsQbEshbnUpukULgWv+77YRZ0lFj3EksXiWr7a+dA6QCE0RbCawYDi zbOQTLi14TLy/n6W9MUAblLgz+mqMDzdbcOh02pzBauBw/FU0uaiT9I9VzBwNIEcTXn3mxJat cBYSa4DenP4qScXcoFUnedFsHvEpTs/4OeYfFyJ7gU7+Q03wEQWbwZZaUJOx5iWfizSFTxoGa l6REq/1FPSedsnP7sG+Vf8N5xbMex9rRoAymnjbnnskIx5NKJg5J8Qw+rirMbUwQhDEtNTSu9 PnpcT4asZ38APZDofEiHvlSrjm0/wWXbx+9Cbl+MFL5orMnJ4zZ0221FaubxdiuWOD32JgwB6 9TU7vJYX+cXM+MZvvRy2596U6HBZXg6ysrGS4XjLnO1WA7N3IWZTDbd2YIf5zdGJVT/xCOY0r mbTa5d64HPcKkrJznmMzl0g+OZg+eiPnCpWC6g== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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: 058dfc767008 ("ACPI / watchdog: Add support for WDAT hardware watchdog") Signed-off-by: Arnd Bergmann --- drivers/acpi/acpi_watchdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/acpi_watchdog.c b/drivers/acpi/acpi_watchdog.c index bf22c29d2517..11b113f8e367 100644 --- a/drivers/acpi/acpi_watchdog.c +++ b/drivers/acpi/acpi_watchdog.c @@ -66,7 +66,7 @@ void __init acpi_watchdog_init(void) for (i = 0; i < wdat->entries; i++) { const struct acpi_generic_address *gas; struct resource_entry *rentry; - struct resource res; + struct resource res = {}; bool found; gas = &entries[i].register_region; -- 2.9.0