linux-kernel - Re: [PATCH RESEND] sst: fix missing breaks that would cause the wrong operation to execute

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]

Message-ID: <5705B844.7050101@linux.intel.com>
Date:	Thu, 7 Apr 2016 09:30:44 +0800
From:	Yang Jie <yang.jie@...ux.intel.com>
To:	Alan <alan@...ux.intel.com>, linux-kernel@...r.kernel.org,
	Vinod Koul <vinod.koul@...el.com>
Subject: Re: [PATCH RESEND] sst: fix missing breaks that would cause the wrong
 operation to execute



On 2016年04月06日 21:44, Alan wrote:
> From: Alan <gnomes@...rguk.ukuu.org.uk>
>
> Now we correctly error an attempt to execute an unsupported operation. The
> current code does something else random.
>
> Signed-off-by: Alan Cox <alan@...ux.intel.com>

I think this is nice fix. + Vinod who is the owner of the atom sound driver.

Thanks,
~Keyon

> ---
>   sound/soc/intel/atom/sst-mfld-platform-compress.c |    9 +++++++--
>   1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/sound/soc/intel/atom/sst-mfld-platform-compress.c b/sound/soc/intel/atom/sst-mfld-platform-compress.c
> index 3951689..1bead81 100644
> --- a/sound/soc/intel/atom/sst-mfld-platform-compress.c
> +++ b/sound/soc/intel/atom/sst-mfld-platform-compress.c
> @@ -182,24 +182,29 @@ static int sst_platform_compr_trigger(struct snd_compr_stream *cstream, int cmd)
>   	case SNDRV_PCM_TRIGGER_START:
>   		if (stream->compr_ops->stream_start)
>   			return stream->compr_ops->stream_start(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_STOP:
>   		if (stream->compr_ops->stream_drop)
>   			return stream->compr_ops->stream_drop(sst->dev, stream->id);
> +		break;
>   	case SND_COMPR_TRIGGER_DRAIN:
>   		if (stream->compr_ops->stream_drain)
>   			return stream->compr_ops->stream_drain(sst->dev, stream->id);
> +		break;
>   	case SND_COMPR_TRIGGER_PARTIAL_DRAIN:
>   		if (stream->compr_ops->stream_partial_drain)
>   			return stream->compr_ops->stream_partial_drain(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
>   		if (stream->compr_ops->stream_pause)
>   			return stream->compr_ops->stream_pause(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
>   		if (stream->compr_ops->stream_pause_release)
>   			return stream->compr_ops->stream_pause_release(sst->dev, stream->id);
> -	default:
> -		return -EINVAL;
> +		break;
>   	}
> +	return -EINVAL;
>   }
>
>   static int sst_platform_compr_pointer(struct snd_compr_stream *cstream,
>
>