linux-kernel - Re: [ 12/37] ALSA: hda - Cirrus: Correctly clear line_out

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]

Date:	Mon, 3 Dec 2012 12:56:45 -0800
From:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:	David Henningsson <david.henningsson@...onical.com>
Cc:	linux-kernel@...r.kernel.org, stable@...r.kernel.org,
	alan@...rguk.ukuu.org.uk, Takashi Iwai <tiwai@...e.de>
Subject: Re: [ 12/37] ALSA: hda - Cirrus: Correctly clear line_out_pins when
 moving to speaker

On Mon, Dec 03, 2012 at 10:46:59AM +0100, David Henningsson wrote:
> On 11/30/2012 07:45 PM, Greg Kroah-Hartman wrote:
> >3.0-stable review patch.  If anyone has any objections, please let me know.
> >
> >------------------
> >
> >From: David Henningsson <david.henningsson@...onical.com>
> >
> >commit 34c3d1926bdaf45d3a891dd577482abcdd9faa34 upstream.
> >
> >If this array is not cleared, the jack related code later might
> >fail to create "Internal Speaker Phantom Jack" on Dell Inspiron 3420 and
> >Dell Vostro 2420.
> >
> >BugLink: https://bugs.launchpad.net/bugs/1076840
> >Signed-off-by: David Henningsson <david.henningsson@...onical.com>
> >Signed-off-by: Takashi Iwai <tiwai@...e.de>
> >Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> >
> >---
> >  sound/pci/hda/patch_cirrus.c |    1 +
> >  1 file changed, 1 insertion(+)
> >
> >--- a/sound/pci/hda/patch_cirrus.c
> >+++ b/sound/pci/hda/patch_cirrus.c
> >@@ -413,6 +413,7 @@ static int parse_output(struct hda_codec
> >  		memcpy(cfg->speaker_pins, cfg->line_out_pins,
> >  		       sizeof(cfg->speaker_pins));
> >  		cfg->line_outs = 0;
> >+		memset(cfg->line_out_pins, 0, sizeof(cfg->line_out_pins));
> >  	}
> >
> >  	return 0;
> >
> >
> 
> This one was originally targeted for 3.6+ (as mentioned in my
> original patch), for earlier kernels I don't anticipate it having
> any effect, neither positive nor negative. Your call.

Ah, I messed up on that, sorry, I've dropped it from the 3.0-stable
queue now.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/