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

Remove multiple subtly different inline and static implementaitonos of ilog()
Add a few additional sanity checks, mostly functioning to ensure we're always
calling ilog() with >=0 input.

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

Add return code check to _initial_pcmoffset as vorbis_packet_blocksize can fail.  Closes Trac #2142

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

Commit patch in Trac #2067
replaces -O20 with -O3 in configure.ac

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

Reorder declaration and initialization of ci and hi so that declaraion is not occurring after the rate <=0 check.

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

Reject multiple headers of the same type.

A common application pattern is to call vorbis_synthesis_headerin()
 and count how many times it succeeds.
If you feed it multiple valid comment headers, they will all
 succeed, meaning you can be fooled into think you have a valid
 Vorbis file despite never seeing a setup header.
This patch makes libvorbis reject multiple headers of the same type,
 preventing this from occurring.

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

Bump libvorbisenc version for 1.3.5 release.

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

Fix encoder crash with non-positive sample rates.

Input files with sampling rate 0 are useless and can make
libvorbis access invalid memory because the logic in
_vp_psy_init (and probably other functions) isn't prepared for
it. A sanity check lets the library refuse those inputs
gracefully in the initialization functions before they can
do harm.

Patch by Martin Steghöfer <martin@steghoefer.eu>

Fixes #2078
Bug-Debian: https://bugs.debian.org/716613

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

Bump version for 1.3.5 release.

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

Fix crash on corrupt input file (invalid mode index)

vorbis_packet_blocksize() crashes with a NULL pointer dereference,
if the "mode" index read from the packet is too large. Check this
immediately after reading the value and before accessing the mode
parameters. This crash potentially affects all users of libvorbisfile
and anyone else who calls vorbis_packet_blocksize() manually.

Patch by Martin Steghöfer <martin@steghoefer.eu>

Fixes #2140
Bug-Debian: https://bugs.debian.org/774516

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

Fix spec typo.

Issue #2060.

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

Fix some typos in the spec.

Thanks to stefan for the report.

Fixes #2060.

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

Correct two more corner case problems with #1486:
1) single-page streams will have no bisection; handle the special case explicitly
2) if the bisection fails without ever successfully grabbing a page, the first-page-special-case check is unprotected and will look at uninitialized memory.

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

Update vorbisfile source copyright

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

Additional fix to last-page handling, this time in initial bisection;
the code had gotten too cavalier about directly munging the vf->offset
file pointer as well as using it as an implicit argument to
_get_prev_page and _get_prev_page_serial.  The fact it was being used
as an argument and a side effect caused a necessary seek to get missed
when the last page of a link was non-vorbis.

Fix: Clean up the overloading, and be explicit about where we're
beginning prev_page reads.

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

Correct two errors in patch for Trac #1486:

1) vf->offset should not have been overridden in new front-of-stream case
2) missing status output in seeking_example.c

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

Attempted fix to Trac #1486

beginning-of-stream seek targets were no longer well defined in
ov_pcm_seek_page() (and thus ov_pcm_seek()) since extending to
multiplexed streams.  Beginning of stream is a special case due to the
seek target preceding the first explicit granule position.

Although seeking to PCM 0 triggered the bug, early seeks in general
were not ebing handld well; rather than continuing to overload the
bisection, handle the early-seek case outside the loop.

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

Extend seeking_example to always perform a pcm / pcm_page seek test to position 0 as an addiitonal test

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

Bump vendor string for dot release.

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

Move all the static encoder codebooks to 'char' lengthlists rather than 'long' as the largest value for a lengthlist entry is 32 and the lengthlists are large.

Alter the codebook generation tools in vq/ to generate the codebooks properly given the new type.

[Hopefully] no functional change.

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

debug and profile targets weren't hooked to anything; have them make check with flags rather than make all

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

Various warning fixes.

Some of these were introduced in r19028.
Others have been around much longer.

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

Make definition of _vorbis_window_get() match declaration.

window.c wasn't actually including window.h, or this would have been
 noticed earlier.

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

Enable keyword substitution on remaining files with $Id$ lines.

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

Add license header and turn on keyword substitution for 5.1 codebook.

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

Set keyword substitution on the 5.1 mode files.

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

Patch for Trac #1974; make sure unit tests pull in -lm during link

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

Patch for trac #1965; clarify in comments and docs that a failed call
to ov_test does not take ownership of the passed-in file.

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

Update pkgconfig files to reduce overlinking.

Patch from Cristian Rodriguez.
Reviewed by: David Schleef, Conrad Parker, Ron Lee
Thanks to Brad Smith for kicking us.
Fixes #1759.

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

Bring libvorbis's configure.in into the 21st century.

Patch by Ron Lee.

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

Escape more things in the vorbisfile docs' example code.

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

corrected docs: docs did not reflect the path parameter to be const. see #1593

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

Have to remember to update CHANGES for release

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

commit version bump for 1.3.3 release

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

Remove white-xifish.png from makefile dependencies for spec; it doesn't appear to
reference it, and trying to install it twice is breaking make install.

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

Minor build correction to r18183

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

Port r17546 from Tremor; although pieces had made it over to libvorbis, a comprehensive
port and verification was called for.  This patch provided some additional floor0
hardening:

  floor0 code could potentially use a book where the number of vals it
  needed to decode was not an integer number of dims wide.  This caused
  it to overflow the output vector as the termination condition was in
  the outer loop of vorbis_book_decodev_set.

  None of the various vorbis_book_decodeXXXX calls internally guard
  against this case either, but in every other use the calling code does
  properly guard (and avoids putting more checks in the tight inner
  decode loop).

  For floor0, move the checks into the inner loop as there's little
  penalty for doing so.

[an equivalent change was already in libvorbis, but I've
harmonized the code with tremor]

  For floor0, move the checks into the inner loop as there's little
  penalty for doing so.  Add commentary indicating where guarding is
  done for each call variant.

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