An uint8_t gets promoted to an int before left shifting, so that it is
subject to the usual restrictions of signed types wrt shifts. Given that
the destination is an unsigned, simply convert to unsigned before
shifting.
Fixes tickets #8163, #8164 and #8165.
Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
---
libavformat/sdsdec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/sdsdec.c b/libavformat/sdsdec.c
index 9c361cdff2..adc7d02a4d 100644
--- a/libavformat/sdsdec.c
+++ b/libavformat/sdsdec.c
@@ -43,7 +43,7 @@ static void byte2_read(const uint8_t *src, uint32_t *dst)
int i;
for (i = 0; i < 120; i += 2) {
- unsigned sample = (src[i + 0] << 25) + (src[i + 1] << 18);
+ unsigned sample = ((unsigned)src[i + 0] << 25) + (src[i + 1] << 18);
dst[i / 2] = sample;
}
@@ -56,7 +56,7 @@ static void byte3_read(const uint8_t *src, uint32_t *dst)
for (i = 0; i < 120; i += 3) {
unsigned sample;
- sample = (src[i + 0] << 25) | (src[i + 1] << 18) | (src[i + 2] << 11);
+ sample = ((unsigned)src[i + 0] << 25) | (src[i + 1] << 18) | (src[i +
2] << 11);
dst[i / 3] = sample;
}
}
@@ -68,7 +68,7 @@ static void byte4_read(const uint8_t *src, uint32_t *dst)
for (i = 0; i < 120; i += 4) {
unsigned sample;
- sample = (src[i + 0] << 25) | (src[i + 1] << 18) | (src[i + 2] << 11)
| (src[i + 3] << 4);
+ sample = ((unsigned)src[i + 0] << 25) | (src[i + 1] << 18) | (src[i +
2] << 11) | (src[i + 3] << 4);
dst[i / 4] = sample;
}
}
--
2.20.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
