gnu: wavpack: Fix CVE-2018-7253 and CVE-2018-7254.
* gnu/packages/patches/wavpack-CVE-2018-7253.patch, gnu/packages/patches/wavpack-CVE-2018-7254.patch: New files. * gnu/local.mk (dist_patch_DATA): Register them. * gnu/packages/audio.scm (wavpack)[source](patches): Use them.
This commit is contained in:
parent
b5bb0e4391
commit
65f704f373
|
@ -1140,6 +1140,8 @@ dist_patch_DATA = \
|
||||||
%D%/packages/patches/vsearch-unbundle-cityhash.patch \
|
%D%/packages/patches/vsearch-unbundle-cityhash.patch \
|
||||||
%D%/packages/patches/vte-CVE-2012-2738-pt1.patch \
|
%D%/packages/patches/vte-CVE-2012-2738-pt1.patch \
|
||||||
%D%/packages/patches/vte-CVE-2012-2738-pt2.patch \
|
%D%/packages/patches/vte-CVE-2012-2738-pt2.patch \
|
||||||
|
%D%/packages/patches/wavpack-CVE-2018-7253.patch \
|
||||||
|
%D%/packages/patches/wavpack-CVE-2018-7254.patch \
|
||||||
%D%/packages/patches/weechat-python.patch \
|
%D%/packages/patches/weechat-python.patch \
|
||||||
%D%/packages/patches/wicd-bitrate-none-fix.patch \
|
%D%/packages/patches/wicd-bitrate-none-fix.patch \
|
||||||
%D%/packages/patches/wicd-get-selected-profile-fix.patch \
|
%D%/packages/patches/wicd-get-selected-profile-fix.patch \
|
||||||
|
|
|
@ -2377,6 +2377,8 @@ stretching and pitch scaling of audio. This package contains the library.")
|
||||||
(method url-fetch)
|
(method url-fetch)
|
||||||
(uri (string-append "http://www.wavpack.com/"
|
(uri (string-append "http://www.wavpack.com/"
|
||||||
name "-" version ".tar.bz2"))
|
name "-" version ".tar.bz2"))
|
||||||
|
(patches (search-patches "wavpack-CVE-2018-7253.patch"
|
||||||
|
"wavpack-CVE-2018-7254.patch"))
|
||||||
(sha256
|
(sha256
|
||||||
(base32
|
(base32
|
||||||
"0i19c6krc0p9krwrqy9s5xahaafigqzxcn31piidmlaqadyn4f8r"))))
|
"0i19c6krc0p9krwrqy9s5xahaafigqzxcn31piidmlaqadyn4f8r"))))
|
||||||
|
|
|
@ -0,0 +1,29 @@
|
||||||
|
Fix CVE-2018-7253:
|
||||||
|
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2018-7253
|
||||||
|
|
||||||
|
Copied from upstream:
|
||||||
|
https://github.com/dbry/WavPack/commit/36a24c7881427d2e1e4dc1cef58f19eee0d13aec
|
||||||
|
|
||||||
|
diff --git a/cli/dsdiff.c b/cli/dsdiff.c
|
||||||
|
index 410dc1c..c016df9 100644
|
||||||
|
--- a/cli/dsdiff.c
|
||||||
|
+++ b/cli/dsdiff.c
|
||||||
|
@@ -153,7 +153,17 @@ int ParseDsdiffHeaderConfig (FILE *infile, char *infilename, char *fourcc, Wavpa
|
||||||
|
error_line ("dsdiff file version = 0x%08x", version);
|
||||||
|
}
|
||||||
|
else if (!strncmp (dff_chunk_header.ckID, "PROP", 4)) {
|
||||||
|
- char *prop_chunk = malloc ((size_t) dff_chunk_header.ckDataSize);
|
||||||
|
+ char *prop_chunk;
|
||||||
|
+
|
||||||
|
+ if (dff_chunk_header.ckDataSize < 4 || dff_chunk_header.ckDataSize > 1024) {
|
||||||
|
+ error_line ("%s is not a valid .DFF file!", infilename);
|
||||||
|
+ return WAVPACK_SOFT_ERROR;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (debug_logging_mode)
|
||||||
|
+ error_line ("got PROP chunk of %d bytes total", (int) dff_chunk_header.ckDataSize);
|
||||||
|
+
|
||||||
|
+ prop_chunk = malloc ((size_t) dff_chunk_header.ckDataSize);
|
||||||
|
|
||||||
|
if (!DoReadFile (infile, prop_chunk, (uint32_t) dff_chunk_header.ckDataSize, &bcount) ||
|
||||||
|
bcount != dff_chunk_header.ckDataSize) {
|
|
@ -0,0 +1,62 @@
|
||||||
|
Fix CVE-2018-7254:
|
||||||
|
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2018-7254
|
||||||
|
|
||||||
|
Copied from upstream:
|
||||||
|
https://github.com/dbry/WavPack/commit/8e3fe45a7bac31d9a3b558ae0079e2d92a04799e
|
||||||
|
|
||||||
|
diff --git a/cli/caff.c b/cli/caff.c
|
||||||
|
index ae57c4b..6248a71 100644
|
||||||
|
--- a/cli/caff.c
|
||||||
|
+++ b/cli/caff.c
|
||||||
|
@@ -89,8 +89,8 @@ typedef struct
|
||||||
|
|
||||||
|
#define CAFChannelDescriptionFormat "LLLLL"
|
||||||
|
|
||||||
|
-static const char TMH_full [] = { 1,2,3,13,9,10,5,6,12,14,15,16,17,9,4,18,7,8,19,20,21 };
|
||||||
|
-static const char TMH_std [] = { 1,2,3,11,8,9,5,6,10,12,13,14,15,7,4,16 };
|
||||||
|
+static const char TMH_full [] = { 1,2,3,13,9,10,5,6,12,14,15,16,17,9,4,18,7,8,19,20,21,0 };
|
||||||
|
+static const char TMH_std [] = { 1,2,3,11,8,9,5,6,10,12,13,14,15,7,4,16,0 };
|
||||||
|
|
||||||
|
static struct {
|
||||||
|
uint32_t mChannelLayoutTag; // Core Audio layout, 100 - 146 in high word, num channels in low word
|
||||||
|
@@ -274,10 +274,19 @@ int ParseCaffHeaderConfig (FILE *infile, char *infilename, char *fourcc, Wavpack
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else if (!strncmp (caf_chunk_header.mChunkType, "chan", 4)) {
|
||||||
|
- CAFChannelLayout *caf_channel_layout = malloc ((size_t) caf_chunk_header.mChunkSize);
|
||||||
|
+ CAFChannelLayout *caf_channel_layout;
|
||||||
|
|
||||||
|
- if (caf_chunk_header.mChunkSize < sizeof (CAFChannelLayout) ||
|
||||||
|
- !DoReadFile (infile, caf_channel_layout, (uint32_t) caf_chunk_header.mChunkSize, &bcount) ||
|
||||||
|
+ if (caf_chunk_header.mChunkSize < sizeof (CAFChannelLayout) || caf_chunk_header.mChunkSize > 1024) {
|
||||||
|
+ error_line ("this .CAF file has an invalid 'chan' chunk!");
|
||||||
|
+ return WAVPACK_SOFT_ERROR;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (debug_logging_mode)
|
||||||
|
+ error_line ("'chan' chunk is %d bytes", (int) caf_chunk_header.mChunkSize);
|
||||||
|
+
|
||||||
|
+ caf_channel_layout = malloc ((size_t) caf_chunk_header.mChunkSize);
|
||||||
|
+
|
||||||
|
+ if (!DoReadFile (infile, caf_channel_layout, (uint32_t) caf_chunk_header.mChunkSize, &bcount) ||
|
||||||
|
bcount != caf_chunk_header.mChunkSize) {
|
||||||
|
error_line ("%s is not a valid .CAF file!", infilename);
|
||||||
|
free (caf_channel_layout);
|
||||||
|
@@ -495,8 +504,15 @@ int ParseCaffHeaderConfig (FILE *infile, char *infilename, char *fourcc, Wavpack
|
||||||
|
}
|
||||||
|
else { // just copy unknown chunks to output file
|
||||||
|
|
||||||
|
- int bytes_to_copy = (uint32_t) caf_chunk_header.mChunkSize;
|
||||||
|
- char *buff = malloc (bytes_to_copy);
|
||||||
|
+ uint32_t bytes_to_copy = (uint32_t) caf_chunk_header.mChunkSize;
|
||||||
|
+ char *buff;
|
||||||
|
+
|
||||||
|
+ if (caf_chunk_header.mChunkSize < 0 || caf_chunk_header.mChunkSize > 1048576) {
|
||||||
|
+ error_line ("%s is not a valid .CAF file!", infilename);
|
||||||
|
+ return WAVPACK_SOFT_ERROR;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ buff = malloc (bytes_to_copy);
|
||||||
|
|
||||||
|
if (debug_logging_mode)
|
||||||
|
error_line ("extra unknown chunk \"%c%c%c%c\" of %d bytes",
|
Loading…
Reference in New Issue