[Skiboot] [PATCH] sensors: dts: Fix uninitialised data in dts result when reading asynchronously
Stewart Smith
stewart at linux.vnet.ibm.com
Tue Mar 27 17:01:26 AEDT 2018
Nicholas Piggin <npiggin at gmail.com> writes:
> dts_async_read_temp() must zero its on-stack dts structure, similarly
> to dts_sensor_read().
>
> Cc: Shilpasri G Bhat <shilpa.bhat at linux.vnet.ibm.com>
> Cc: Cédric Le Goater <clg at kaod.org>
> Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
> ---
>
> The used uninitialized warning showed up when I tried to compile
> skiboot with -O2. Too bad -Os and other static checkers didn't
> catch it.
Any idea what gcc gave it? I think the warning is wrong, as we have the
struct being initialized to zero:
static void dts_async_read_temp(struct timer *t __unused, void *data,
u64 now __unused)
{
struct dts dts = {0};
which, according to K&R should mean all members are zeroed.
Or am I suitably jetlagged and missing something?
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list