diff options
author | naddy <naddy@FreeBSD.org> | 2015-09-11 03:46:30 +0800 |
---|---|---|
committer | naddy <naddy@FreeBSD.org> | 2015-09-11 03:46:30 +0800 |
commit | 1be7f62fa6db48f587fbb66acc95630290e2fb6b (patch) | |
tree | a12c50a5594dadeb8a1d6b8ce3ba7603e93e32a2 /audio | |
parent | 6eea0a44a6d3230486980f164591082134c18dda (diff) | |
download | freebsd-ports-graphics-1be7f62fa6db48f587fbb66acc95630290e2fb6b.tar.gz freebsd-ports-graphics-1be7f62fa6db48f587fbb66acc95630290e2fb6b.tar.zst freebsd-ports-graphics-1be7f62fa6db48f587fbb66acc95630290e2fb6b.zip |
Fix opusenc buffer overflow, channel integer overflow, and division
by zero. (Same code as vorbis-tools oggenc.)
PR: 202941
Obtained from: https://trac.xiph.org/ticket/2212
Obtained from: https://trac.xiph.org/changeset/19117
Obtained from: Fedora vorbis-tools Git (commit 63a1a62d)
Security: CVE-2015-6749
Security: CVE-2014-9638
Security: CVE-2014-9639
Security: a35f415d-572a-11e5-b0a4-f8b156b6dcc8
MFH: 2015Q3
Diffstat (limited to 'audio')
-rw-r--r-- | audio/opus-tools/Makefile | 2 | ||||
-rw-r--r-- | audio/opus-tools/files/patch-src_audio-in.c | 85 |
2 files changed, 86 insertions, 1 deletions
diff --git a/audio/opus-tools/Makefile b/audio/opus-tools/Makefile index 3c731542aa3..08135710a0f 100644 --- a/audio/opus-tools/Makefile +++ b/audio/opus-tools/Makefile @@ -2,7 +2,7 @@ PORTNAME= opus-tools PORTVERSION= 0.1.9 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= audio MASTER_SITES= http://downloads.xiph.org/releases/opus/ \ MOZILLA/opus diff --git a/audio/opus-tools/files/patch-src_audio-in.c b/audio/opus-tools/files/patch-src_audio-in.c new file mode 100644 index 00000000000..8c72337c9c3 --- /dev/null +++ b/audio/opus-tools/files/patch-src_audio-in.c @@ -0,0 +1,85 @@ +--- src/audio-in.c.orig 2014-02-26 00:55:47 UTC ++++ src/audio-in.c +@@ -42,6 +42,7 @@ + # define _FILE_OFFSET_BITS 64 + #endif + ++#include <limits.h> + #include <stdlib.h> + #include <stdio.h> + #include <string.h> +@@ -287,13 +288,14 @@ static int aiff_permute_matrix[6][6] = + int aiff_open(FILE *in, oe_enc_opt *opt, unsigned char *buf, int buflen) + { + int aifc; /* AIFC or AIFF? */ +- unsigned int len; +- unsigned char *buffer; ++ unsigned int len, readlen; ++ unsigned char buffer[22]; + unsigned char buf2[8]; + int bigendian = 1; + aiff_fmt format; + aifffile *aiff; + int i; ++ long channels; + (void)buflen;/*unused*/ + + if(buf[11]=='C') +@@ -313,19 +315,25 @@ int aiff_open(FILE *in, oe_enc_opt *opt, + return 0; /* Weird common chunk */ + } + +- buffer = alloca(len); +- +- if(fread(buffer,1,len,in) < len) ++ readlen = len < sizeof(buffer) ? len : sizeof(buffer); ++ if(fread(buffer,1,readlen,in) < readlen || ++ (len > readlen && !seek_forward(in, len-readlen))) + { + fprintf(stderr, _("Warning: Unexpected EOF reading AIFF header\n")); + return 0; + } + +- format.channels = READ_U16_BE(buffer); ++ format.channels = channels = READ_U16_BE(buffer); + format.totalframes = READ_U32_BE(buffer+2); + format.samplesize = READ_U16_BE(buffer+6); + format.rate = (int)read_IEEE80(buffer+8); + ++ if(channels <= 0L || SHRT_MAX < channels) ++ { ++ fprintf(stderr, _("Warning: Unsupported count of channels in AIFF header\n")); ++ return 0; ++ } ++ + if(aifc) + { + if(len < 22) +@@ -442,6 +450,7 @@ int wav_open(FILE *in, oe_enc_opt *opt, + wav_fmt format; + wavfile *wav; + int i; ++ long channels; + (void)buflen;/*unused*/ + (void)oldbuf;/*unused*/ + +@@ -481,12 +490,18 @@ int wav_open(FILE *in, oe_enc_opt *opt, + } + + format.format = READ_U16_LE(buf); +- format.channels = READ_U16_LE(buf+2); ++ format.channels = channels = READ_U16_LE(buf+2); + format.samplerate = READ_U32_LE(buf+4); + format.bytespersec = READ_U32_LE(buf+8); + format.align = READ_U16_LE(buf+12); + format.samplesize = READ_U16_LE(buf+14); + ++ if(channels <= 0L || SHRT_MAX < channels) ++ { ++ fprintf(stderr, _("Warning: Unsupported count of channels in WAV header\n")); ++ return 0; ++ } ++ + if(format.format == -2) /* WAVE_FORMAT_EXTENSIBLE */ + { + if(len<40) |