From be663d3b8ea26471966589164532442277bb0ca1 Mon Sep 17 00:00:00 2001
From: spiderpig <26628298+spiderpigpig@users.noreply.github.com>
Date: Mon, 31 Mar 2025 14:35:06 +0800
Subject: [PATCH] Fix: Align destination width to 32 for hardware decoder
 compatibility

- Modified the calculation of `dw` to use `FFALIGN(sw, 32)` instead of `sw >> 2 << 2`.
- Ensured that the destination width is aligned to a multiple of 32, which improves compatibility with hardware decoders (e.g., h264_cuvid).
- This change resolves potential crashes caused by misaligned resolutions when using `sws_scale`.

Related logs:
- Original resolution (888x550) caused crashes due to unaligned width.
- After alignment, the issue was resolved and the program runs stably."
---
 src/video/hffplayer.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/video/hffplayer.cpp b/src/video/hffplayer.cpp
index b6dfbf2..2435858 100644
--- a/src/video/hffplayer.cpp
+++ b/src/video/hffplayer.cpp
@@ -233,7 +233,8 @@ int HFFPlayer::open() {
         return ret;
     }
 
-    dw = sw >> 2 << 2; // align = 4
+    // dw = sw >> 2 << 2; // align = 4
+    dw = FFALIGN(sw, 32);
     dh = sh;
     dst_pix_fmt = AV_PIX_FMT_YUV420P;
     std::string str = g_confile->GetValue("dst_pix_fmt", "video");