From 4c920ce777a1904202952ea3010fdfb99e002ad3 Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sat, 13 Feb 2016 15:32:58 +0100 Subject: [PATCH] avutil/imgutils: only align the palette in av_image_copy_to_buffer() if there is enough space This allows disabling the alignment by using a compact buffer Reviewed-by: Stefano Sabatini Signed-off-by: Michael Niedermayer --- libavutil/imgutils.c | 5 +++++ libavutil/version.h | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c index 5c2870323c..adf6fdd206 100644 --- a/libavutil/imgutils.c +++ b/libavutil/imgutils.c @@ -385,6 +385,7 @@ int av_image_copy_to_buffer(uint8_t *dst, int dst_size, int i, j, nb_planes = 0, linesize[4]; int size = av_image_get_buffer_size(pix_fmt, width, height, align); const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(pix_fmt); + uint8_t *orig_dst = dst; if (size > dst_size || size < 0 || !desc) return AVERROR(EINVAL); @@ -409,6 +410,10 @@ int av_image_copy_to_buffer(uint8_t *dst, int dst_size, if (desc->flags & AV_PIX_FMT_FLAG_PAL) { uint32_t *d32 = (uint32_t *)(((size_t)dst + 3) & ~3); + + if (dst_size - 1024 < (uint8_t*)d32 - orig_dst) + d32 = (uint32_t *)dst; + for (i = 0; i<256; i++) AV_WL32(d32 + i, AV_RN32(src_data[1] + 4*i)); } diff --git a/libavutil/version.h b/libavutil/version.h index f99624c94b..aa7d9137a7 100644 --- a/libavutil/version.h +++ b/libavutil/version.h @@ -65,7 +65,7 @@ #define LIBAVUTIL_VERSION_MAJOR 55 #define LIBAVUTIL_VERSION_MINOR 17 -#define LIBAVUTIL_VERSION_MICRO 101 +#define LIBAVUTIL_VERSION_MICRO 102 #define LIBAVUTIL_VERSION_INT AV_VERSION_INT(LIBAVUTIL_VERSION_MAJOR, \ LIBAVUTIL_VERSION_MINOR, \