[FFmpeg-devel] [PATCH 1/2] lavc/vorbisdec: use ptrdiff_t to iterate over intptr_t

remi at remlab.net remi at remlab.net
Sat Sep 17 12:59:35 EEST 2022


From: Rémi Denis-Courmont <remi at remlab.net>

While this probably never overflows, we are better safe than sorry.

The callback prototype should probably also use ptrdiff_t or size_t but
I  diggress.
---
 libavcodec/vorbisdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c
index 38a5367be3..9728997370 100644
--- a/libavcodec/vorbisdec.c
+++ b/libavcodec/vorbisdec.c
@@ -1581,7 +1581,7 @@ static inline int vorbis_residue_decode(vorbis_context *vc, vorbis_residue *vr,
 
 void ff_vorbis_inverse_coupling(float *mag, float *ang, intptr_t blocksize)
 {
-    int i;
+    ptrdiff_t i;
     for (i = 0;  i < blocksize;  i++) {
         if (mag[i] > 0.0) {
             if (ang[i] > 0.0) {
-- 
2.37.2



More information about the ffmpeg-devel mailing list