- Jan 20, 2017
-
-
Without the fix, very large NLSF values could cause the stabilization code in silk/NLSF_stabilize.c to wrap-around and have the last value in NLSF_Q15[] to be negative, close to -32768. That value would then be used in silk_NLSF2A() to compute f_int, which would be equal to -128. Since f_int is used to look up into constant table silk_LSFCosTab_FIX_Q12[], it would cause two 16-bit reads, 256 bytes and 254 bytes before the constant table. In nornal circumstances the code will simply read from the wrong table, resulting in an unstable LPC filter. The filter would then go through the LPC stabilization code at the end of silk_NLSF2A(). Ultimately the output audio would be garbage, but no worse than with any other harmless bad packet. For this bug to cause a crash, the linker would have to put the relevant at the very beginning of the segment, with unaddressable memory just before it. Alternatively, if the code is compiled with assertions enable, then it will abort. The only way this could cause a data leak would be for the linker to put the silk_LSFCosTab_FIX_Q12[] within 256 bytes after sensitive process information, which is highly unlikely. Even in that circumstance, only 32 bits of data could be read, at location outside of the attacker's control. The output would be in the form of audio that would have to be mapped back to the original 32-bit data. This was reported as CVE-2017-0381. Contrary to that report, we do not believe that any remote code execution is possible. Signed-off-by:
Jean-Marc Valin <jmvalin@jmvalin.ca>
- Jul 15, 2016
-
-
Jean-Marc Valin authored
No values outside of +/-1 detected now.
-
Jean-Marc Valin authored
-
Jean-Marc Valin authored
-
- Jul 08, 2016
-
-
Notably, ARM floating-point build. This maintains the invariant that we don't use later instruction sets if the OS claims an earlier one is not available. However, it does not update configure to ensure that there is build support for all earlier instruction sets if NEON build support is enabled (though I am not aware of a build toolchain where this is actually a problem). Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Jean-Marc Valin authored
-
Jean-Marc Valin authored
-
Mark Harris authored
-
Ralph Giles authored
Add a matrix build testing the most common options. I don't see a way to also invoke Makefile.unix without wrapping everything in a script, so leave that to jenkins for now.
-
Ralph Giles authored
-
- Jul 07, 2016
-
-
Jean-Marc Valin authored
-
Jean-Marc Valin authored
-
Jean-Marc Valin authored
-
Jean-Marc Valin authored
len<0 still returns OPUS_BAD_ARG
-
Jean-Marc Valin authored
Rather than read invalid memory
-
- Jul 06, 2016
-
-
Timothy B. Terriberry authored
There are no tabs in source code.
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
This makes it match the formatting of the output for ARM assembly better, and removes some redundant repetition of the word "intrinsics". It also fixes the output if a compiler supports RTCD for Neon intrinsics but not assembly. Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Used for celt_pitch_xcorr on aarch64, and celt_fir and celt_iir on both armv7 and aarch64. Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Enables existing Neon intrinsic optimizations to work on aarch64 targets. Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Signed-off-by:
Timothy B. Terriberry <tterribe@xiph.org>
-
Mark Harris authored
-
Ralph Giles authored
Thanks to RiCON for reporting this.
-
Ralph Giles authored
-
Ralph Giles authored
-