alsa: limit buffer_size to 32768 frames.
authorJustin Ruggles <justin.ruggles@gmail.com>
Wed, 22 Jun 2011 20:38:20 +0000 (16:38 -0400)
committerJustin Ruggles <justin.ruggles@gmail.com>
Sat, 9 Jul 2011 20:01:06 +0000 (16:01 -0400)
In testing, the file output plugin gave a max buffer size of about 20 million
frames, which is way more than what is really needed and causes a memory
allocation error on my system.

libavdevice/alsa-audio-common.c
libavdevice/alsa-audio.h

index 1e945fc..126695f 100644 (file)
@@ -260,6 +260,7 @@ av_cold int ff_alsa_open(AVFormatContext *ctx, snd_pcm_stream_t mode,
     }
 
     snd_pcm_hw_params_get_buffer_size_max(hw_params, &buffer_size);
+    buffer_size = FFMIN(buffer_size, ALSA_BUFFER_SIZE_MAX);
     /* TODO: maybe use ctx->max_picture_buffer somehow */
     res = snd_pcm_hw_params_set_buffer_size_near(h, hw_params, &buffer_size);
     if (res < 0) {
index 1e0be1c..ee43463 100644 (file)
@@ -40,6 +40,8 @@
         other formats */
 #define DEFAULT_CODEC_ID AV_NE(CODEC_ID_PCM_S16BE, CODEC_ID_PCM_S16LE)
 
+#define ALSA_BUFFER_SIZE_MAX 32768
+
 typedef struct {
     AVClass *class;
     snd_pcm_t *h;