linux-kernel - Re: [PATCH v3] soundwire: debugfs: move debug statement outside of error handling

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] [day] [month] [year] [list]

Message-Id: <175097827732.85045.8539937079296009855.b4-ty@kernel.org>
Date: Thu, 26 Jun 2025 15:51:17 -0700
From: Vinod Koul <vkoul@...nel.org>
To: yung-chuan.liao@...ux.intel.com, pierre-louis.bossart@...ux.dev, 
 sanyog.r.kale@...el.com, Rodrigo Gobbi <rodrigo.gobbi.7@...il.com>
Cc: ~lkcamp/patches@...ts.sr.ht, linux-sound@...r.kernel.org, 
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] soundwire: debugfs: move debug statement outside of
 error handling


On Thu, 26 Jun 2025 18:33:14 -0300, Rodrigo Gobbi wrote:
> The start_t and finish_t variables are not properly initialized
> if errors happens over request_firmware actions.
> This was also detected by smatch:
> 
> drivers/soundwire/debugfs.c:301 cmd_go() error: uninitialized symbol 'finish_t'.
> drivers/soundwire/debugfs.c:301 cmd_go() error: uninitialized symbol 'start_t'.
> 
> [...]

Applied, thanks!

[1/1] soundwire: debugfs: move debug statement outside of error handling
      commit: 06f77ff9d852c9f2764659ea81489364d8a69a9c

Best regards,
-- 
~Vinod