crec: Always read whole number of fragments when length limit specified

This patch removes the lines that adjusted the read size to exactly
read to the end of the calculated length in bytes. Doing this could
lead to invalid fragment sizes being requested which may then cause
a read error on some codecs.

Instead we keep reading whole fragments until we have >= length bytes
of data captured.

Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>

diff --git a/src/utils/crec.c b/src/utils/crec.c
index 258fadee51b813f8a5ba246c195832c6edb62713..e61e620d649093ef793c8a28b607f7b9c09708d8 100644 (file)
--- a/src/utils/crec.c
+++ b/src/utils/crec.c
@@ -332,9 +332,6 @@ void capture_samples(char *name, unsigned int card, unsigned int device,
                fprintf(finfo, "%s: Capturing audio NOW!!!\n", __func__);
 
        do {
-               if (length && size > length - total_read)
-                       size = length - total_read;
-
                read = compress_read(compress, buffer, size);
                if (read < 0) {
                        fprintf(stderr, "Error reading sample\n");