components/sox/patches/9.CVE-2014-8145.patch
author Norm Jacobs <Norm.Jacobs@Oracle.COM>
Fri, 09 Jan 2015 10:58:01 -0800
branchs11u2-sru
changeset 3627 4d299d6b2a9a
permissions -rw-r--r--
20332735 problem in UTILITY/SOX
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
3627
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     1
#
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     2
# Fix CVE-2014-8145[0]:
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     3
#    two heap-based buffer overflows
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     4
# Upstream changesets 7d3f38 and f39c57.
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     5
# http://sourceforge.net/p/sox/code/ci/7d3f38007a1eeaf5ab7669aba0d2a7e4d3def57e/
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     6
# http://sourceforge.net/p/sox/code/ci/f39c574bc423fd5b12bd6510264512f5d5366183/
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     7
#
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     8
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
     9
--- a/src/sphere.c
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    10
+++ b/src/sphere.c
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    11
@@ -47,6 +47,11 @@
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    12
 
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    13
   /* Determine header size, and allocate a buffer large enough to hold it. */
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    14
   sscanf(fldsval, "%lu", &header_size_ul);
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    15
+  if (header_size_ul < 16) {
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    16
+    lsx_fail_errno(ft, SOX_EHDR, "Error reading Sphere header");
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    17
+    return (SOX_EOF);
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    18
+  }
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    19
+
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    20
   buf = lsx_malloc(header_size = header_size_ul);
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    21
 
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    22
   /* Skip what we have read so far */
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    23
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    24
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    25
--- a/src/wav.c
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    26
+++ b/src/wav.c
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    27
@@ -166,7 +166,7 @@
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    28
         /* work with partial blocks.  Specs say it should be null */
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    29
         /* padded but I guess this is better than trailing quiet. */
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    30
         samplesThisBlock = lsx_ms_adpcm_samples_in((size_t)0, (size_t)ft->signal.channels, bytesRead, (size_t)0);
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    31
-        if (samplesThisBlock == 0)
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    32
+        if (samplesThisBlock == 0 || samplesThisBlock > wav->samplesPerBlock)
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    33
         {
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    34
             lsx_warn("Premature EOF on .wav input file");
4d299d6b2a9a 20332735 problem in UTILITY/SOX
Norm Jacobs <Norm.Jacobs@Oracle.COM>
parents:
diff changeset
    35
             return 0;