[PATCH] resampler: Fix crash if 'auto' resampler chooses ffmpeg with variable rate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2012-08-22 at 15:42 +0200, Fr?d?ric Dalleau wrote:
[...]
> diff --git a/src/pulsecore/resampler.c b/src/pulsecore/resampler.c
> index 17f1783..8199bc4 100644
> --- a/src/pulsecore/resampler.c
> +++ b/src/pulsecore/resampler.c
> @@ -237,7 +237,10 @@ pa_resampler* pa_resampler_new(
>  #ifdef HAVE_SPEEX
>          method = PA_RESAMPLER_SPEEX_FLOAT_BASE + 3;
>  #else
> -        method = PA_RESAMPLER_FFMPEG;
> +        if (flags & PA_RESAMPLER_VARIABLE_RATE)
> +            method = PA_RESAMPLER_TRIVIAL;
> +        else
> +            method = PA_RESAMPLER_FFMPEG;
>  #endif
>      }

The trivial resampler is not a good fallback. I'd rather make speex a
mandatory dep again.

-- Arun



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux