Such calls as "copy_CTB(dst - (left_pixels << sh), ..." may actually
use completely unaligned addresses, causing issues on platforms such
as ARM where 64bits copies have alignment requirements.

In such cases, AC_COPY64U should be use instead.

-- 
Christophe
From 030105f7b0978e18fd45e54dd0ed56d79f3f7332 Mon Sep 17 00:00:00 2001
From: Christophe Gisquet <christophe.gisq...@gmail.com>
Date: Mon, 2 Mar 2015 18:17:28 +0100
Subject: [PATCH] hevc/sao: use unaligned copy

---
 libavcodec/hevc_filter.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/hevc_filter.c b/libavcodec/hevc_filter.c
index 855a619..f50a640 100644
--- a/libavcodec/hevc_filter.c
+++ b/libavcodec/hevc_filter.c
@@ -147,7 +147,7 @@ int i, j;
     if (((intptr_t)dst | (intptr_t)src | stride_dst | stride_src) & 15) {
         for (i = 0; i < height; i++) {
             for (j = 0; j < width; j+=8)
-                AV_COPY64(dst+j, src+j);
+                AV_COPY64U(dst+j, src+j);
             dst += stride_dst;
             src += stride_src;
         }
-- 
1.9.2.msysgit.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to