1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20
|
Description: wav: fix crash writing header when channel count >64k
High number of channels (>64k) lead to divide-by-zero error and crash. Number
of channels should be representable with 16 bits, so forbid any higher value.
Author: Mans Rullgard <mans@mansr.com>
Origin: upstream, https://github.com/mansr/sox/commit/8b590b3a52f4ccc4eea3f41b4a067c38b3565b60
--- a/src/wav.c 2019-03-05 17:05:44.053925697 +0100
+++ b/src/wav.c 2019-03-05 17:07:53.657036855 +0100
@@ -1278,6 +1278,12 @@
long blocksWritten = 0;
sox_bool isExtensible = sox_false; /* WAVE_FORMAT_EXTENSIBLE? */
+ if (ft->signal.channels > UINT16_MAX) {
+ lsx_fail_errno(ft, SOX_EOF, "Too many channels (%u)",
+ ft->signal.channels);
+ return SOX_EOF;
+ }
+
dwSamplesPerSecond = ft->signal.rate;
wChannels = ft->signal.channels;
wBitsPerSample = ft->encoding.bits_per_sample;
|