Commit 5f9b488216 for ffmpeg
commit 5f9b4882166651da0515bbd0b074e71137545717
Author: Timo Rothenpieler <timo@rothenpieler.org>
Date: Wed Oct 7 02:25:56 2026 +0200
swscale/aarch64: fix uyvy/yuyv to yuv420p with a height of 1
For yuv420 output, interleaved_yuv_to_planar processes two source lines
per iteration and keeps the pair count, height >> 1, in w5. Both the
fast and the slow path test that counter only at the bottom of the loop,
so a height of 1 starts with w5 == 0: the body runs once, reading a
second source line and writing a second luma line that don't exist,
then w5 wraps to -1 and the loop runs far past the buffers.
A height of 1 is reached with 1-line frames, and also when swscale
slice-threads a conversion and the last slice is a single line.
Skip the pair loop when there are no full line pairs and go straight to
the existing odd-last-line handling. The fast-path tail code reads the
tail width from w9, which is otherwise only set at the loop head, so set
it there too.
uyvytoyuv422 and yuyvtoyuv422 do not pair lines and are not affected.
Fixes: #YWH-PGM45796-1
Reported-by: jpraveenrao
Assisted-by: Claude Opus 5.5
diff --git a/libswscale/aarch64/rgb2rgb_neon.S b/libswscale/aarch64/rgb2rgb_neon.S
index ba2f904879..76a9e07774 100644
--- a/libswscale/aarch64/rgb2rgb_neon.S
+++ b/libswscale/aarch64/rgb2rgb_neon.S
@@ -884,6 +884,9 @@ function ff_\src_fmt\()to\dst_fmt\()_neon, export=1
b.eq 6f
+.ifc \dst_fmt, yuv420
+ cbz w5, 30f // height==1: no full line pairs, handle the lone line only
+.endif
1: // fast path - the width is at least 32
and w14, w4, #~31 // w14 is the main loop counter
and w9, w4, #31 // w9 holds the remaining width, 0 to 31
@@ -898,7 +901,9 @@ function ff_\src_fmt\()to\dst_fmt\()_neon, export=1
b.ne 1b
.ifc \dst_fmt, yuv420 // handle the last line in case the height is odd
+30:
cbz w17, 3f
+ and w9, w4, #31 // tail width, needed by the shift-back below
and w14, w4, #~31
4:
fastpath_iteration \src_fmt, \dst_fmt, 0, 1
@@ -916,6 +921,9 @@ function ff_\src_fmt\()to\dst_fmt\()_neon, export=1
6: // slow path - width is at most 31
and w9, w4, #31
+.ifc \dst_fmt, yuv420
+ cbz w5, 60f // height==1: no full line pairs, handle the lone line only
+.endif
cbz w9, 9f // even part empty (orig width 0 or 1)
7:
subs w9, w9, #2
@@ -928,6 +936,7 @@ function ff_\src_fmt\()to\dst_fmt\()_neon, export=1
b.ne 6b
.ifc \dst_fmt, yuv420
+60:
cbz w17, 8f
and w9, w4, #31
.ifc \src_fmt, uyvy