[alsa-devel] [PATCH] ALSA: compress: fix the states to check for allowing read

[Vinod Koul]

for reading compressed data, we need to allow when we are paused, draining or
stopped.

Signed-off-by: Vinod Koul <vinod.koul at intel.com>
Cc: Charles Keepax <ckeepax at opensource.wolfsonmicro.com>
Cc: Richard Fitzgerald <rf at opensource.wolfsonmicro.com>

---
 sound/core/compress_offload.c |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c
index a0bc47f..5389b9a 100644
--- a/sound/core/compress_offload.c
+++ b/sound/core/compress_offload.c
@@ -311,8 +311,14 @@ static ssize_t snd_compr_read(struct file *f, char __user *buf,
 	stream = &data->stream;
 	mutex_lock(&stream->device->lock);
 
-	/* read is allowed when stream is running */
-	if (stream->runtime->state != SNDRV_PCM_STATE_RUNNING) {
+	/* read is allowed when stream is running, paused, draining and setup
+	 * (yes setup is state which we transistion to after stop, so if user
+	 * wants to read data after stop we allow that
+	 */
+	if (stream->runtime->state != SNDRV_PCM_STATE_RUNNING ||
+			stream->runtime->state != SNDRV_PCM_STATE_DRAINING ||
+			stream->runtime->state != SNDRV_PCM_STATE_PAUSED ||
+			stream->runtime->state != SNDRV_PCM_STATE_SETUP) {
 		retval = -EBADFD;
 		goto out;
 	}
-- 
1.7.0.4