[FFmpeg-devel] [PATCH] libavutil/ppc: defines involving bit shifts should be unsigned

Sean McGovern gseanmcg at gmail.com
Sat Nov 2 08:13:05 EET 2024


Otherwise, these can overflow at the boundaries of the integer type.
---
 libavutil/ppc/cpu.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libavutil/ppc/cpu.c b/libavutil/ppc/cpu.c
index 62d495ec1d..9f9c073434 100644
--- a/libavutil/ppc/cpu.c
+++ b/libavutil/ppc/cpu.c
@@ -53,9 +53,9 @@
 #define AT_HWCAP2            26
 #endif
 
-#define HWCAP_PPC_VSX        (1 << 7)
-#define HWCAP_PPC_ALTIVEC    (1 << 28)
-#define HWCAP2_PPC_ARCH_2_07 (1 << 31)
+#define HWCAP_PPC_VSX        (1U << 7)
+#define HWCAP_PPC_ALTIVEC    (1U << 28)
+#define HWCAP2_PPC_ARCH_2_07 (1U << 31)
 
 /**
  * This function MAY rely on signal() or fork() in order to make sure AltiVec
-- 
2.45.2



More information about the ffmpeg-devel mailing list