Update CMake links.

Use secure links to the default domain and current documentation.

Signed-off-by: Mark Harris <mark.hsj@gmail.com>

Update ISRC link.

The current link redirects to this secure link.

Signed-off-by: Mark Harris <mark.hsj@gmail.com>

Update website link in copyright headers.

Use the secure link, which should be the default. Prefer the
url without the `www` prefix because it's shorter.

Signed-off-by: Mark Harris <mark.hsj@gmail.com>

Add further array bounds checks to bark_noise_hybridmp.

Make it clear to local analysis that no out-of-bounds array
accesses are possible here.

Follow-up to CVE-2018-10393 and CVE-2017-14160.

Signed-off-by: Thomas Daede <daede003@umn.edu>
Signed-off-by: Monty <xiphmont@xiph.org>

autotools: Use AC_CANONICAL_HOST

We were calling AC_CANONICAL_TARGET in configure.ac under the
mistaken impression that this allowed setting the target
architecture for cross-compilation output.

However, in GNU terminology the system type that this the
target of cross-compilation is the "host" and "target"
is reserved for setting what the output of the compiled
application should itself target, when compiling a
cross-compiler. Finally "build" is the system type the
build system is itself running under.

So to cross-compile, one invokes `./configure --host ...`
not `--target`.

In any case, we only make use of the `host` variable,
so that is the macro we need to invoke to provide it.

Thanks to Maarten Bosmans for reporting the issue.

Signed-off-by: Marvin Scholz <epirat07@gmail.com>

pkg-config: Only append -lm if necessary.

Some systems don't require a separate -lm to link with libc
math functions. The configure script checks for this, so
use the corresponding variable to set the dependent libraries
in vorbis.pc dynamically.

Signed-off-by: Marvin Scholz <epirat07@gmail.com>

gitlab-ci: Use the gcc:9 base image.

Use a versioned base image for jobs to reduce variance.

This version number will need to be bumped periodically.

Signed-off-by: Marvin Scholz <epirat07@gmail.com>

gitlab-ci: Don't cache object files.

This could only help with the initial build, not `make distcheck`
and since config.h is generated fresh for each job, everything
was remade anyway. Thanks to ePirat for pointing out the issue.

gitlab-ci: Move shared tags to a default section.

Move shared properties into a `default` stanza per current
style recommendations.

Add cmake build to gitlab ci.

Add an additional build job to the gitlab ci pipeline to do a
cmake build. This doesn't run tests, but gives us a little
bit of converage.

Add gitlab ci configuration.

Describe a basic autoconf build and test for gitlab's integrated
continuous integration runner. Uses the gcc docker image.

Copied from the theora version.

- `zip` is needed for `make dist`
- `doxygen` is needed to generate api documentation.
  latex is also needed, but a heavy dependency to install every run,
  so skipped for now.

Instead of SIGFPE when bytespersample is zero or negative, return OV_EINVAL

Based on0002-Avoid-SIGFPE-when-bytespersample-is-zero.patch from Debian,
see https://bugs.debian.org/635906 , adjusted with input from Ralph
Giles.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Silence some CMake build MSVC useless warnings

Supress warnings about unsafe and deprecated functions like this: strcat is unsafe, use `strcat_s` instead and so on.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Export public function on MinGW platform

MinGW produces DLLs, but exports nothing.

Fix CMake include install directory

Fix CMake config-file package generation

travis-ci: Build on the xcode11 macOS image.

Fix the macOS builds on the travis-ci.org continuous integration
system by requiring a more recently-created system image where
homebrew packages install correctly.

Also switch to declarative syntax for the homebrew package
dependencies, using the built-in Brewfile support instead
of invoking `brew` explicitly.

Travis doesn't update the homebrew three on their default macOS
images, so over time available packages become out-of-date, or
any explicit update step takes longer and longer, slowing the
testing feedback loop.

In this particular instance jobs were failing because homebrew
on the default macOS image wasn't working at all. Without an
update, package installation failed because the `brew bundle`
subcommand had itself bitrotted, while running `brew update`
would time out, taking more than the allowed job time.

Requiring a more recent (non-default) macOS image gets out
tests working again. In the future this will probably need
to be bumped again or restored to the default image.

For more context, see https://travis-ci.community/t/macos-build-fails-because-of-homebrew-bundle-unknown-command/7296/14

win32: Update VS2010 project files for ogg library name.

The ogg project changed the default target, making libogg.lib
a static library instead of libogg_static.lib. Update our
build to match.

The VS2005 and VS2008 projects are left as-is, because
I have no way to test them and they likely aren't in
active use.

appveyor: Use the correct ogg solution file.

In ogg commit 6d55ddf64b65, the static build was made the
default, removing the separate static target solution file,
breaking our build on the Appveyor continuous integration system.

Clean up trailing whitespace.

Rewrap EXTRA_DIST lines.

Make the grouping a little more logical.

Add CMake config-file package generation

Signed-off-by: Ralph Giles <giles@thaumas.net>

oss-fuzz: replace -lFuzzingEngine with $LIB_FUZZING_ENGINE

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Don't leak memory in test_sharedbook to please asan.

os.h: make vorbis_ftoi() static inline if !VORBIS_FPU_CONTROL

os.h: cleanup the _MSC_VER cpp checks mess.

fix typo introduced in commit 8d73daa

Check for sse2 math before using it.

info.c (tagcompare): use a locale-insensitive toupper()

see: https://gitlab.xiph.org/xiph/vorbis/issues/2079

Port opus-tools check to see if underlying file was overwritten.

Fix free of uninitialized memory if seek fails in ov_raw_seek

If _seek_helper fails in ov_raw_seek, control jumps to the seek_error
label which calls ogg_stream_clear on work_os. However, at this point
in the function, work_os is not initialized so we end up attempting to
free some uninitialized memory and crashing.

Fix by removing the call to ogg_stream_clear. This is safe because the
only code path to seek_error happens before work_os is initialized (so
there is never anything to free anyway).

I also refactor the code a bit:
- Remove the ret variable which is unnessesary since we can just pass
  the result of _seek_helper directly to the if.
- Since seek_error is only used once, move the contents of that block
  to the if statement so we can remove a goto.

Remove C99-style comments.

Fix the compiling errors on msvc ARM64 configuration.

Fix shift by negative value when reading blocksize.

add osx to travis

add fuzzers and build script from OSS-Fuzz

Sanity check number of channels in setup.

Fixes #2335.

Clamp large exponents in float32_unpack.

CVE-2017-14160: fix bounds check on very low sample rates.

vorbisenc: detect if new_template is NULL

Fixes #1975

Include CMake build scripts in release archives

Fixes xiph/vorbis#32

Update release version in configure.ac to 1.3.6

Update CHANGES and dates for 1.3.6.

Bump version to 1.3.6

CVE-2018-5146: Prevent out-of-bounds write in codebook decoding.

Codebooks that are not an exact divisor of the partition size are now
truncated to fit within the partition.

CVE-2017-14632: vorbis_analysis_header_out: Don't clear opb if not initialized

If the number of channels is not within the allowed range
we call oggback_writeclear altough it's not initialized yet.

This fixes

    =23371== Invalid free() / delete / delete[] / realloc()
    ==23371==    at 0x4C2CE1B: free (vg_replace_malloc.c:530)
    ==23371==    by 0x829CA31: oggpack_writeclear (in /usr/lib/x86_64-linux-gnu/libogg.so.0.8.2)
    ==23371==    by 0x84B96EE: vorbis_analysis_headerout (info.c:652)
    ==23371==    by 0x9FBCBCC: ??? (in /usr/lib/x86_64-linux-gnu/sox/libsox_fmt_vorbis.so)
    ==23371==    by 0x4E524F1: ??? (in /usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
    ==23371==    by 0x4E52CCA: sox_open_write (in /usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
    ==23371==    by 0x10D82A: open_output_file (sox.c:1556)
    ==23371==    by 0x10D82A: process (sox.c:1753)
    ==23371==    by 0x10D82A: main (sox.c:3012)
    ==23371==  Address 0x68768c8 is 488 bytes inside a block of size 880 alloc'd
    ==23371==    at 0x4C2BB1F: malloc (vg_replace_malloc.c:298)
    ==23371==    by 0x4C2DE9F: realloc (vg_replace_malloc.c:785)
    ==23371==    by 0x4E545C2: lsx_realloc (in /usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
    ==23371==    by 0x9FBC9A0: ??? (in /usr/lib/x86_64-linux-gnu/sox/libsox_fmt_vorbis.so)
    ==23371==    by 0x4E524F1: ??? (in /usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
    ==23371==    by 0x4E52CCA: sox_open_write (in /usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
    ==23371==    by 0x10D82A: open_output_file (sox.c:1556)
    ==23371==    by 0x10D82A: process (sox.c:1753)
    ==23371==    by 0x10D82A: main (sox.c:3012)

as seen when using the testcase from CVE-2017-11333 with
008d23b782be09c8d75ba8190b1794abd66c7121 applied. However the error was
there before.

CVE-2017-14633: Don't allow for more than 256 channels

Otherwise

 for(i=0;i<vi->channels;i++){
      /* the encoder setup assumes that all the modes used by any
         specific bitrate tweaking use the same floor */
      int submap=info->chmuxlist[i];

overreads later in mapping0_forward since chmuxlist is a fixed array of
256 elements max.

Fixed error in Vorbis I specification for limiting residue vector size

The minimum between the encoded residue boundaries and actual block size
needs to be used, otherwise it pushes the boundaries to the edge of the
actual blocksize or beyond.

vorbis: avoid invalid free

Regression started at commit "4b67376 Remove multiple subtly different inline..."

Add build flags for YouCompleteMe.

Add a .ycm_extra.conf.py script to return the same CFLAGS
we pass for `make debug`. These are passed to libclang
so symbol lookup works correctly.

Note this doesn't pick up changes to the build config,
including non-default locations for the ogg headers,
but it's better than nothing.

mapping0.c (mapping0_unpack): kill a useless memset()

info is allocated with calloc() already.

Signed-off-by: Ralph Giles <giles@thaumas.net>

os.h: change elif _WIN32 to elif defined(_WIN32)

This symbol is only defined (with the value 1) when building
for the Windows target, so we need to ifdef, not if.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Cleanup: Removed tailing white-spaces in C code files

Signed-off-by: Ralph Giles <giles@thaumas.net>

Update release checklist for the transition to git.

The tagging method is different now that our main
repository is in git instead of subversion.

CMake: Add vorbisenc.c to vorbis target for WIN32 builds

On Windows the vorbis shared library exports some symbols defined in the
vorbisen.c file.  To allow error free linking this file should be added
to to the VORBIS_SOURCES.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Fix reading maximum, nominal, minimum bitrate in _vorbis_unpack_info().

https://xiph.org/vorbis/doc/Vorbis_I_spec.html#x1-630004.2.2 specifies
these fields as 32bit signed. oggpack_read(opb,32), which is used to
read these fields, returns 32 bits stored in a long. On architectures
where long is 64bit, this results in a positive value being returned.
This value is then stored in struct vorbis_info in bitrate_upper,
bitrate_nominal and bitrate_lower, also as long. ogginfo relies on
these values in order to display the respective header fields and thus
misrepresented the stored value -1 (which has the intended meaning of
"bitrate not set") as 2**32-1 on architectures where long is 64bit.

Explicitly cast the return value of oggpack_read() to a signed 32bit
integer type.

A nominal bitrate value of -1 is valid as per specification, and is
written by libvorbis for VBR files with samplerate >= 50000Hz.

Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>

Remove svn $Id$ header.

Most checked-in files had a comment with a filename and
last-modified string automatically updated by the
subversion version control tool. These became obsolete
when we migrated the repository to git. Remove them.

Fix signed overflow in _book_maptype1_quantvals().

If b->dims is very large, vals will be 1 and acc1 will overflow a
 long.
This causes us to read too many values for this codebook.

Signed-off-by: Monty <xiphmont@xiph.org>

Add AppVeyor build status badge.

Make CMake build test on Travis-CI mandatory

Signed-off-by: Ralph Giles <giles@thaumas.net>

Add cmake as optional build system to Travis-CI configuration

Add BUILD_SYSTEM env variable to Travis-CI build matrix

This allows to switch between different build systems when building
multiple test matrix configurations.

autotools: Fix doc_DATA install.

Both the generated format specification document and the static
html documents depend on the logo in fish_xiph_org.png.

Unfortunately we can't list them both in their respective
dependencies because both are passed to doc_DATA for installation
and automake doesn't seem to remove duplicates, causing
/usr/bin/install to complain about the overwrite.

I couldn't find a good solution to this. Wrapping the doc_DATA
value in GNU make's $(sort /list/) function would work, but
automake warns about the non-portable construct. Instead, I've
just removed it from the spec dependency list and appended it
manually.

Add missing graphics for the html spec.

New equations were added to the spec source in the previous
update (early last year) but the generated graphic versions
where not added to the repository.

Also add all the generated figures to the distribution. No
sense in shipping a spec with broken links.

Make CMake build test on AppVeyor mandatory

Signed-off-by: Ralph Giles <giles@thaumas.net>

Use different install dir for libogg dependency in CMake AppVeyor

Add OGG_ROOT search path to AppVeyor CMake build

Build CMake install target for libogg dependency on AppVeyor

Add cmake as optional build system to AppVeyor configuration

Add BUILD_SYSTEM env variable to AppVeyor build matrix

This allows to switch between different build systems when building
multiple test matrix configurations.

Use build_script step in AppVeyor configuration

Preparations for building with multiple build systems

Fix CMake error

Include vorbis library when linking vorbisfile and vorbisenc.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Initial appveyor config to test the Windows build.

This makes the minimal changes necessary to build
the dynamic solution Win32|Debug target with VS2015
and against the static solution from the ogg master
branch.

The other targets can be added after updating the
project files for VS2015.

Add ids to v-comment.html

This allows linking to specific sections.

Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>

doc/Makefile.am: Fix `clean` target

The `doc/Vorbis_I_spec*.png` files are checked into git, but were also
being incorrectly deleted during `make clean` because they were listed
in `CLEANFILES`.

Update doxygen message.

Configure only looks for doxygen if --enable-docs is passed,
so report that as the issue instead.

Pass -Wall -Wextra when building on linux32.

We want to be warning-free, so this should be the default,
not just debug builds. It already is on most other targets.

Make travis builds verbose.

Recover the full compiler command lines for reference in the
logs after the AM_SILENT_RULES change.

Enable AM_SILENT_RULES.

Makes warnings more obvious.

Fix install of pkgconfig files and allow use of OGG_ROOT variable

Use svg image for TravisCI badge

Enable parallel builds on TravisCI

Add testing with clang compiler

travis: use sudo-less apt-plugin install of libogg-dev.

This lets our test job dispatch on the container infrastructure,
which is faster.

Allocate comment temporaries on the heap.

Use malloc/free instead of the more convenient alloca for
comment data. Album art can easily be larger than the local
stack limit and crash the process.

Thanks to Robert Kausch for the suggestion.

Remove spurious variable from autogen.sh.

configure.ac: Add AC_CONFIG_MACRO_DIR

configure.ac: Whitespace only

Add CMake documentation to readme.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Add CMake script to build vorbis.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Use https urls in README.

Remove trailing whitespace.

Add build description for travis continuous integration service.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Switch README to markdown.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Remove AC_CONFIG_MACRO_DIRS.

Jenkins doesn't have new enough libtool for this.

Make git ignore generated files.

Fix automake warning.

Newer versions prefer AM_CPPFLAGS to INCLUDES.
Tell libtool about our m4 directory.

Simplify autogen.sh

Copy the autogen.sh script from the newer daala project. This
just calls autoreconf instead of doing its own tool and
version detection.

Note that this now means one must explicitly invoke ./configure
as ./autogen.sh will no longer do so.

Remove todo and MacOS 9 files from distribution.

Fixes 'make distcheck'.

Remove Mac OS 9 project files

Signed-off-by: Ralph Giles <giles@thaumas.net>