* [PATCH] ALSA: drivers: opl3: Fix incorrect use of vp->state
@ 2021-12-12 17:20 Colin Ian King
2021-12-13 9:36 ` Takashi Iwai
0 siblings, 1 reply; 2+ messages in thread
From: Colin Ian King @ 2021-12-12 17:20 UTC (permalink / raw)
To: Jaroslav Kysela, Takashi Iwai, Pierre-Louis Bossart, alsa-devel
Cc: kernel-janitors, linux-kernel
Static analysis with scan-build has found an assignment to vp2 that is
never used. It seems that the check on vp->state > 0 should be actually
on vp2->state instead. Fix this.
This dates back to 2002, I found the offending commit from the git
history git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git,
commit 91e39521bbf6 ("[PATCH] ALSA patch for 2.5.4")
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
sound/drivers/opl3/opl3_midi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/drivers/opl3/opl3_midi.c b/sound/drivers/opl3/opl3_midi.c
index e1b69c65c3c8..e2b7be67f0e3 100644
--- a/sound/drivers/opl3/opl3_midi.c
+++ b/sound/drivers/opl3/opl3_midi.c
@@ -397,7 +397,7 @@ void snd_opl3_note_on(void *p, int note, int vel, struct snd_midi_channel *chan)
}
if (instr_4op) {
vp2 = &opl3->voices[voice + 3];
- if (vp->state > 0) {
+ if (vp2->state > 0) {
opl3_reg = reg_side | (OPL3_REG_KEYON_BLOCK +
voice_offset + 3);
reg_val = vp->keyon_reg & ~OPL3_KEYON_BIT;
--
2.33.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ALSA: drivers: opl3: Fix incorrect use of vp->state
2021-12-12 17:20 [PATCH] ALSA: drivers: opl3: Fix incorrect use of vp->state Colin Ian King
@ 2021-12-13 9:36 ` Takashi Iwai
0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2021-12-13 9:36 UTC (permalink / raw)
To: Colin Ian King
Cc: alsa-devel, linux-kernel, kernel-janitors, Pierre-Louis Bossart,
Takashi Iwai
On Sun, 12 Dec 2021 18:20:25 +0100,
Colin Ian King wrote:
>
> Static analysis with scan-build has found an assignment to vp2 that is
> never used. It seems that the check on vp->state > 0 should be actually
> on vp2->state instead. Fix this.
>
> This dates back to 2002, I found the offending commit from the git
> history git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git,
> commit 91e39521bbf6 ("[PATCH] ALSA patch for 2.5.4")
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Thanks, applied now with Cc-to-stable.
Takashi
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-13 9:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-12 17:20 [PATCH] ALSA: drivers: opl3: Fix incorrect use of vp->state Colin Ian King
2021-12-13 9:36 ` Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).