At Wed, 15 May 2013 19:11:47 +0200, Jordi Mallach wrote:
The a52 plugin uses av_free(), but does not check for libavutil availability. Patch configure.in to check for the module, and include libavutil.h.
Signed-off-by: Jordi Mallach jordi@debian.org
Well, the unconditional inclusion of libavutil/avutil.h is not 100% correct fix, but it should be OK for 99% of systems right now, so I took it as is.
thanks,
Takashi
a52/pcm_a52.c | 1 + configure.in | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/a52/pcm_a52.c b/a52/pcm_a52.c index c40aadc..75c68bd 100644 --- a/a52/pcm_a52.c +++ b/a52/pcm_a52.c @@ -26,6 +26,7 @@ #include <alsa/pcm_external.h> #include <alsa/pcm_plugin.h> #include AVCODEC_HEADER +#include <libavutil/avutil.h>
#if LIBAVCODEC_VERSION_MAJOR >= 53 && LIBAVCODEC_VERSION_MINOR >= 34 #include <libavutil/audioconvert.h> diff --git a/configure.in b/configure.in index 6f345f5..bf8676b 100644 --- a/configure.in +++ b/configure.in @@ -67,7 +67,7 @@ AC_ARG_ENABLE([avcodec], AS_HELP_STRING([--disable-avcodec], [Don't build plugins depending on avcodec (a52)]))
if test "x$enable_avcodec" != "xno"; then
- PKG_CHECK_MODULES(AVCODEC, [libavcodec], [HAVE_AVCODEC=yes], [HAVE_AVCODEC=no])
- PKG_CHECK_MODULES(AVCODEC, [libavcodec libavutil], [HAVE_AVCODEC=yes], [HAVE_AVCODEC=no])
fi
if test "x$HAVE_AVCODEC" = "xno"; then
1.7.10.4
Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel