Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 49b02fd9 authored by Viraja Kommaraju's avatar Viraja Kommaraju Committed by Gerrit - the friendly Code Review server
Browse files

ASoC: Add check for pointer and array index 



Add check for NULL pointer access and array index boundary.

Change-Id: Ia12a28c43b99158a90d27eecd23bdfd4b42b346e
Signed-off-by: default avatarViraja Kommaraju <virajak@codeaurora.org>
Signed-off-by: default avatarMeng Wang <mwang@codeaurora.org>
parent 8736b065

include/sound/pcm_params.h

+8 −4
Original line number Diff line number Diff line
@@ -110,10 +110,14 @@ static inline void snd_mask_reset_range(struct snd_mask *mask, 


static inline void snd_mask_leave(struct snd_mask *mask, unsigned int val)

{

	unsigned int v;

	v = mask->bits[MASK_OFS(val)] & MASK_BIT(val);

	unsigned int v, bits_index;



	bits_index = MASK_OFS(val);

	if (bits_index < ((SNDRV_MASK_MAX+31)/32)) {

		v = mask->bits[bits_index] & MASK_BIT(val);

		snd_mask_none(mask);

	mask->bits[MASK_OFS(val)] = v;

		mask->bits[bits_index] = v;

	}

}



static inline void snd_mask_intersect(struct snd_mask *mask,

sound/soc/soc-pcm.c

+1 −1
Original line number Diff line number Diff line
@@ -632,7 +632,7 @@ static int soc_pcm_open(struct snd_pcm_substream *substream) 
		platform->driver->ops->close(substream);



platform_err:

	if (cpu_dai->driver->ops->shutdown)

	if (cpu_dai->driver->ops && cpu_dai->driver->ops->shutdown)

		cpu_dai->driver->ops->shutdown(substream, cpu_dai);

out:

	mutex_unlock(&rtd->pcm_mutex);