[alsa-devel] [PATCH][tinycompress] crecord: Use whole buffer requested by user
Currently, crecord will only actually use 1 fragment worth of its internal buffer no matter what the user requests. Correct this so that the whole buffer requested by the user is used for copying data.
Signed-off-by: Charles Keepax ckeepax@opensource.cirrus.com --- src/utils/crecord.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/utils/crecord.c b/src/utils/crecord.c index e051c2d..8728c5d 100644 --- a/src/utils/crecord.c +++ b/src/utils/crecord.c @@ -370,15 +370,15 @@ static void capture_samples(char *name, unsigned int card, unsigned int device, if (verbose) fprintf(finfo, "%s: Opened compress device\n", __func__);
- size = config.fragment_size; - buffer = malloc(size * config.fragments); + size = config.fragments * config.fragment_size; + buffer = malloc(size); if (!buffer) { fprintf(stderr, "Unable to allocate %d bytes\n", size); goto comp_exit; }
fprintf(finfo, "Recording file %s On Card %u device %u, with buffer of %lu bytes\n", - name, card, device, buffer_size); + name, card, device, size); fprintf(finfo, "Codec %u Format %u Channels %u, %u Hz\n", codec.id, codec.format, codec.ch_out, rate);
On Fri, Apr 13, 2018 at 12:58:59PM +0100, Charles Keepax wrote:
Currently, crecord will only actually use 1 fragment worth of its internal buffer no matter what the user requests. Correct this so that the whole buffer requested by the user is used for copying data.
Applied, thanks
participants (2)
-
Charles Keepax
-
Vinod Koul