On Wed, Aug 4, 2021 at 10:24 AM Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > The "ret" variable is used without being initialized when "alt_path" is > false. > > Fixes: 5ff013914c62 ("brcmfmac: firmware: Allow per-board firmware binaries") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Thanks Dan! But yesterday I sent another patch which incidentally fixes this too: https://lore.kernel.org/linux-wireless/20210803232746.3389570-1-linus.walleij@xxxxxxxxxx/T/#u As you can see in the last hunk of that patch ret is now unconditionally assigned on both forks of the if/else-clause. Yours, Linus Walleij