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>

Link all of the examples to libogg.

More release checklist additions.

svn path=/trunk/vorbis/; revision=19466

Clarify xiph.org updates on the release checklist.

Thanks to mark4o for the suggestions.

svn path=/trunk/vorbis/; revision=19462

Add a release checklist.

I always forget things, this is good documentation, and it's been
helpful with the opus releases.

svn path=/trunk/vorbis/; revision=19461

Record the release date.

Also two more copyright bumps which somehow didn't make the earlier commit.

svn path=/trunk/vorbis/; revision=19460

Bump more copyright dates.

svn path=/trunk/vorbis/; revision=19457

Update changes for release.

svn path=/trunk/vorbis/; revision=19456

Suppress spurious 'may be used uninitialized' gcc warnings in floor1.c
encode with static initializers.

svn path=/trunk/vorbis/; revision=19455

Roll back unintentionally chaged file (barkmel.c) from previous commit.

svn path=/trunk/vorbis/; revision=19454

Remove a set-but-unused variable (lastdelta) in the encode-side lsp
code.  Note that the code in question is effectively dead as it's for
floor0 encoding, and floor zero has been deprecated for over a decade.

svn path=/trunk/vorbis/; revision=19453

Somehow the previous svn rm lspmap.png didn't take.

svn path=/trunk/vorbis/; revision=19451

Errata to correct typo in bark equation in 6.2.5 of spec; the last
paren was misplaced.  The error apparently appeared when resetting the
doc to XML or DocBook.  Note that lspmap.png was erroneously updated
to reflect the typo in r8547 and so it's been wrong since then too.
The original HTML spec was correct (as is the code).

Also remove the associated rendered versions of the typeset equations
as they're no longer used (and lspmap.png was incorrect as described
above anyway)

svn path=/trunk/vorbis/; revision=19450

Don't use MSVC inline assembly on arm targets.

Patch from Hugo Beauzée-Luyssen. Fixes the build in WindowsRT/ARM.

svn path=/trunk/vorbis/; revision=19449

Update docs makefile according to reflect newly removed graphics files.

svn path=/trunk/vorbis/; revision=19448

Remove the ancient (if unlinked) doc/vorbis.html as well as more newly unused graphics.

svn path=/trunk/vorbis/; revision=19447

Eliminate old, neglected DocBook XML version of Vorbis spec doc.  It's
not been consistently maintained n a number of years and does not
contain all additions/corrections made to the TeX.

svn path=/trunk/vorbis/; revision=19446

This commit has two 'functional changes':

Update the footer to use a modern logo and not kick out a formatting error

handle errata discussed in Trac #2139 regarding single-entry codebooks

svn path=/trunk/vorbis/; revision=19445

Trac 2139 indirectly brought to light the case of a stream that uses a
single-entry codebook, but does not code a codeword of length 1 equal
to zero.  Such a stream could cause a stream to read garbage.

There is no apparent chance of garbage memory writes as this happen
entirely after decode setup, however there is playback DoS potential.

This commit special cases single-entry codebook setup so that decode
is well-defined for streams with single-entry codebooks, and adds some
comments to make it more clear how the case is handled.

svn path=/trunk/vorbis/; revision=19444

Render prototype and actual function declatation of ov_ilog() semantically consistent

svn path=/trunk/vorbis/; revision=19442