Takashi Iwai
2ba3b200ab
- Apply various fix patches taken from Debian package; it fixes also other entries (CVE-2022-31650 bsc#1212060 CVE-2023-34318 bsc#1212062 CVE-2023-34432 bsc#1212063): CVE-2019-13590.patch CVE-2021-3643.patch CVE-2021-23159.patch CVE-2021-33844.patch CVE-2021-40426.patch CVE-2022-31650.patch CVE-2022-31651.patch - Fix floating point exception in src/voc.c (CVE-2023-32627 bsc#1212061): CVE-2023-32627.patch OBS-URL: https://build.opensuse.org/request/show/1120234 OBS-URL: https://build.opensuse.org/package/show/multimedia:apps/sox?expand=0&rev=45
47 lines
1.6 KiB
Diff
47 lines
1.6 KiB
Diff
From: Helmut Grohne <helmut@subdivi.de>
|
|
Subject: formats+aiff: reject implausibly large number of channels
|
|
Bug: https://sourceforge.net/p/sox/bugs/360/
|
|
Bug-Debian: https://bugs.debian.org/1012516
|
|
|
|
--- a/src/formats_i.c
|
|
+++ b/src/formats_i.c
|
|
@@ -19,6 +19,7 @@
|
|
*/
|
|
|
|
#include "sox_i.h"
|
|
+#include <limits.h>
|
|
#include <string.h>
|
|
#include <sys/stat.h>
|
|
#include <stdarg.h>
|
|
@@ -60,9 +61,14 @@
|
|
if (ft->seekable)
|
|
ft->data_start = lsx_tell(ft);
|
|
|
|
- if (channels && ft->signal.channels && ft->signal.channels != channels)
|
|
+ if (channels && ft->signal.channels && ft->signal.channels != channels) {
|
|
lsx_warn("`%s': overriding number of channels", ft->filename);
|
|
- else ft->signal.channels = channels;
|
|
+ } else if (channels > SHRT_MAX) {
|
|
+ lsx_fail_errno(ft, EINVAL, "implausibly large number of channels");
|
|
+ return SOX_EOF;
|
|
+ } else {
|
|
+ ft->signal.channels = channels;
|
|
+ }
|
|
|
|
if (rate && ft->signal.rate && ft->signal.rate != rate)
|
|
lsx_warn("`%s': overriding sample rate", ft->filename);
|
|
--- sox-14.4.2+git20190427.orig/src/aiff.c
|
|
+++ sox-14.4.2+git20190427/src/aiff.c
|
|
@@ -609,6 +609,11 @@
|
|
At 48 kHz, 16 bits stereo, this gives ~3 hours of audio.
|
|
Sorry, the AIFF format does not provide for an indefinite
|
|
number of samples. */
|
|
+ if (ft->signal.channels >= (0x7f000000 / (ft->encoding.bits_per_sample >> 3)))
|
|
+ {
|
|
+ lsx_fail_errno(ft, SOX_EOF, "too many channels for AIFF header");
|
|
+ return SOX_EOF;
|
|
+ }
|
|
return(aiffwriteheader(ft, (uint64_t) 0x7f000000 / ((ft->encoding.bits_per_sample>>3)*ft->signal.channels)));
|
|
}
|
|
|