linux-kernel - Re: [PATCH v2 1/2] ASoC: codec: wm9860: avoid maybe-uninitialized warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK8P3a1ZGTyjhAzy4h2D10+-dFNi6o_qe0pY9Wx5wYXZK6ObDw@mail.gmail.com>
Date:   Thu, 27 Apr 2017 12:07:05 +0200
From:   Arnd Bergmann <arnd@...db.de>
To:     Daniel Baluta <daniel.baluta@....com>
Cc:     Mark Brown <broonie@...nel.org>, Takashi Iwai <tiwai@...e.com>,
        Charles Keepax <ckeepax@...nsource.wolfsonmicro.com>,
        Liam Girdwood <lgirdwood@...il.com>,
        patches@...nsource.wolfsonmicro.com, alsa-devel@...a-project.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/2] ASoC: codec: wm9860: avoid maybe-uninitialized warning

On Wed, Apr 26, 2017 at 3:09 PM, Daniel Baluta <daniel.baluta@....com> wrote:
> The new PLL configuration code triggers a harmless warning:
>
> sound/soc/codecs/wm8960.c: In function 'wm8960_configure_clocking':
> sound/soc/codecs/wm8960.c:735:3: error: 'best_freq_out' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
>    wm8960_set_pll(codec, freq_in, best_freq_out);
>    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> sound/soc/codecs/wm8960.c:699:12: note: 'best_freq_out' was declared
> here
>
> Fix this by reworking the code such that:
>
> 1) When there is no PLL freq available return -EINVAL and make
> sure *bclk_idx, *dac_idx, *sysclk_idx are initialized with
> invalid values.
>
> 2) When there is a PLL freq available initialize *bclk_idx,
> *dac_idx and *sysclk_idx with correct values and immediately
> return the freq available.
>
> Fixes: 84fdc00d519f ("ASoC: codec: wm9860: Refactor PLL out freq search")
> Fixes: 303e8954af8d ("ASoC: codec: wm8960: Stop when a matching PLL freq is found")
> Suggested-by: Arnd Bergmann <arnd@...db.de>
> Signed-off-by: Daniel Baluta <daniel.baluta@....com>

Looks good to me. Let me know if you need me to test this more thoroughly
with randconfig builds, otherwise I'll assume it's fine and it will
get tested once
it shows up in linux-next.

Acked-by: Arnd Bergmann <arnd@...db.de>

Thanks for addressing this,

       Arnd