All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync()
@ 2011-01-11 11:24 Dimitris Papastamos
  2011-01-11 11:45 ` Liam Girdwood
  2011-01-11 18:26 ` Mark Brown
  0 siblings, 2 replies; 3+ messages in thread
From: Dimitris Papastamos @ 2011-01-11 11:24 UTC (permalink / raw)
  To: Mark Brown, Liam Girdwood; +Cc: alsa-devel, patches

The size of the lzo syncing bitmap was incorrectly set to the size
of the cache times the word size, however, the correct size is the
size of the cache.

Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
---
 sound/soc/soc-cache.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/soc/soc-cache.c b/sound/soc/soc-cache.c
index a9ebc07..8c2a21a 100644
--- a/sound/soc/soc-cache.c
+++ b/sound/soc/soc-cache.c
@@ -1361,7 +1361,7 @@ static int snd_soc_lzo_cache_init(struct snd_soc_codec *codec)
 			goto err;
 		}
 		lzo_blocks[i]->sync_bmp = sync_bmp;
-		lzo_blocks[i]->sync_bmp_nbits = reg_size;
+		lzo_blocks[i]->sync_bmp_nbits = bmp_size;
 		/* alloc the working space for the compressed block */
 		ret = snd_soc_lzo_prepare(lzo_blocks[i]);
 		if (ret < 0)
-- 
1.7.3.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync()
  2011-01-11 11:24 [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync() Dimitris Papastamos
@ 2011-01-11 11:45 ` Liam Girdwood
  2011-01-11 18:26 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Liam Girdwood @ 2011-01-11 11:45 UTC (permalink / raw)
  To: Dimitris Papastamos; +Cc: alsa-devel, Mark Brown, patches

On Tue, 2011-01-11 at 11:24 +0000, Dimitris Papastamos wrote:
> The size of the lzo syncing bitmap was incorrectly set to the size
> of the cache times the word size, however, the correct size is the
> size of the cache.
> 
> Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
> ---
>  sound/soc/soc-cache.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/sound/soc/soc-cache.c b/sound/soc/soc-cache.c
> index a9ebc07..8c2a21a 100644
> --- a/sound/soc/soc-cache.c
> +++ b/sound/soc/soc-cache.c
> @@ -1361,7 +1361,7 @@ static int snd_soc_lzo_cache_init(struct snd_soc_codec *codec)
>  			goto err;
>  		}
>  		lzo_blocks[i]->sync_bmp = sync_bmp;
> -		lzo_blocks[i]->sync_bmp_nbits = reg_size;
> +		lzo_blocks[i]->sync_bmp_nbits = bmp_size;
>  		/* alloc the working space for the compressed block */
>  		ret = snd_soc_lzo_prepare(lzo_blocks[i]);
>  		if (ret < 0)

Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync()
  2011-01-11 11:24 [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync() Dimitris Papastamos
  2011-01-11 11:45 ` Liam Girdwood
@ 2011-01-11 18:26 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2011-01-11 18:26 UTC (permalink / raw)
  To: Dimitris Papastamos; +Cc: alsa-devel, patches, Liam Girdwood

On Tue, Jan 11, 2011 at 11:24:02AM +0000, Dimitris Papastamos wrote:
> The size of the lzo syncing bitmap was incorrectly set to the size
> of the cache times the word size, however, the correct size is the
> size of the cache.
> 
> Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>

Applied, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-01-11 18:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-11 11:24 [PATCH] ASoC: soc-cache: Fix invalid memory access during snd_soc_lzo_cache_sync() Dimitris Papastamos
2011-01-11 11:45 ` Liam Girdwood
2011-01-11 18:26 ` Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.