linux-kernel - Re: [PATCH] Fixed typo on line 733.

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 for Android: free password hash cracker in your pocket

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

Date:   Thu, 28 Jun 2018 07:39:52 +0200
From:   Takashi Iwai <tiwai@...e.de>
To:     "Guy Chronister" <guylovesbritt@...il.com>
Cc:     <guychronister@...look.com>, <alsa-devel@...a-project.org>,
        "Clemens Ladisch" <clemens@...isch.de>,
        "Jaroslav Kysela" <perex@...ex.cz>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Fixed typo on line 733.

On Wed, 27 Jun 2018 20:54:34 +0200,
Guy Chronister wrote:
> 
> Signed-off-by: Guy Chronister <guylovesbritt@...il.com>

The code change itself is good, but the subject is wrong, and there is
no changelog texts.  Please describe what it really fixes.  It's not
about typo fix, but it fixes a real bug, and this has to be written in
the subject and explained in more details in the changelog.


thanks,

Takashi

> ---
>  sound/pci/oxygen/oxygen_mixer.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sound/pci/oxygen/oxygen_mixer.c b/sound/pci/oxygen/oxygen_mixer.c
> index 81af21a..ba87213 100644
> --- a/sound/pci/oxygen/oxygen_mixer.c
> +++ b/sound/pci/oxygen/oxygen_mixer.c
> @@ -730,7 +730,7 @@ static int ac97_fp_rec_volume_put(struct snd_kcontrol *ctl,
>  	oldreg = oxygen_read_ac97(chip, 1, AC97_REC_GAIN);
>  	newreg = oldreg & ~0x0707;
>  	newreg = newreg | (value->value.integer.value[0] & 7);
> -	newreg = newreg | ((value->value.integer.value[0] & 7) << 8);
> +	newreg = newreg | ((value->value.integer.value[1] & 7) << 8);
>  	change = newreg != oldreg;
>  	if (change)
>  		oxygen_write_ac97(chip, 1, AC97_REC_GAIN, newreg);
> -- 
> 2.7.4
> 
>