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

More typo/layout fixes to the floor 1 spec (packet decode and curve computation ->
subsubsections, fix my escpaed square brackets)

Commit new html/pdf

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

Add a clarification to the floor1 decode spec, since I had to re-determine some of what
it says for myself after not thinking about it for 15-ish years.

Also, fix an indentation bug that occurred due to TABs sneaking in

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

Fix handling of floor0 codebook with no used entries.

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

Commit new versions of geenrated docs to reflect floor1 spec addition/correction

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

Commit fix to Moz Bugzilla #719612 (along with spec update in previous commit)

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

Spec correction: Hard limit of 63 configurable (65 total) floor1 posts was never spelled out in the spec.
Part of addressing Moz BZ #719612

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

Fix a memory leak in vorbis_commentheader_out().

Fixes #1840.
Thanks to Alexandru Munteanu for the patch.

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

Ifdef out an unused variable.

Corrects a warning on Apple gcc 4.2.

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

Add even more parentheses to silence warnings.

This is a more aggressive fix than r17688. This version is from
a similar patch to the Mozilla tree by Atul Aggarwal. See
https://bugzilla.mozilla.org/show_bug.cgi?id=666672 for the
history.

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

Remove generated Makefile.in inadvertently included in r18064.

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

Ngh, whitespace lines are significant in package import

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

Fix numbering in interrupted enumerate lists (eg, in floor0 curve
computation pseudocode)
Regenerate docs with updated/corrected fancvrb package

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

More underscore cleanups; it appears that \usepackage{underscore} in
unreliable in several environments like \mathtt and \emph

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

Hack to deal with the fact that underscores in a \mathtt{} were showing
up as the letter 'X', but using \texttt{} instead caused incorrect type
scaling in math typesetting

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

If we're committing a pregenerated HTML version of the spec, we need to
commit the pregenned equation images with it.

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

Commit corrected/pregenerated spec docs.  This doesn't commit the
pregen images, those come next...

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

The XML had the p/q equations for even LSP curve computation wrong too

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

The DocBook->TeX conversion typoed the even LSP curve computation
equations for p and q; removed the spurious ^2.

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

Fixed project configuration which referenced two CRT versions in output binaries.

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

A trio of warning fixes.

Thanks to Josh Soref for the report.

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

Prefer dist-xz to dist-bzip2. This requires automake 1.11.

The newer xz compression utility offers superiour compression
efficiency *and* speed over bzip2. Since adoption of bzip2
never became near universal, we still provide gzip for those
on older or more conservative systems. Furthermore, for smaller
archives the performance penalty often doesn't pay for the savings
in transfer time we might as well replace the 'smaller' bzip2
option with xz, which at least does better in this regard.

Since Automake 1.11 isn't available in all environments yes
(e.g. some scratchbox toolchains) we leave the old Makefile.am
AUTOMAKE_OPTIONS line commented out. Just uncomment this to
build on systems with older automakes.

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

Remove the generated Makefile.in, inadvertently included in the last commit.

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

Add new API documenation files from the 1.3.2 release.

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

Update vorbis CHANGES file

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

Doc patch removed superfluous files, now need to remove them in SVN

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

Update configure, spec and pc file versions for release

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

Commit new version/vendor string in preparation for release.

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

Update documentation versioning/copyrights toward release

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

It's 'Xiph.Org' not 'Xiph.Org' or 'xIpH.ORg'

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

Commit libvorbis documentation patch from John Stumpo; adds complete
docs to libvorbis, moves a few pages from the vorbisfile and
libvorbisenc docs into the new libvorbis doc tree.

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

libvorbis version of fix for Trac #1578

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

Bump versioning toward upcoming release

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

Back out the old HF AoTuV weighting which was first enabled in
Everywhere.  There are a few samples where I really don't like the
effect it causes.

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

Commit change to libvorbis that returns correct timestamp for granule
positions with high bit set.

Make two 'better safe than sorry' changes in vorbisfile that clamp
pcmoffsets entries to being strictly >=0.

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

Commit tterribe's port of Tremor r17541:
Harden the code that trims the last packet of a stream; it was
possible to game the granpos such that the trim code would try to
rewind more samples than were actually available in storage.

Also, fix/eliminate two printf warnings in seeking_example extension.

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

While reviewing a patch to port Tremor r17541, noticed that the
half-rate decode code in Vorbisfile was never completed past the
original 'quick and dirty' feature request.  Specifically, it made no
attempt to keep the pcm offset tracking consistent in seeks.

Complete this code, and add a testing mode to seeking_example.c to
torture test seeking in halfrate mode.  Also remove requirement that
halfrate mode only work with seekable files.

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

Port r17547 from Tremor.

Disallow books with dim<1 in floor0.

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

Port r17545 from Tremor.

Check that floor0 books all have a value mapping.

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

Port r17544 from Tremor.

A guard conditional used > instead of >=.

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

Port r17543 from Tremor.

A codebook that allows values outside the circular range of the piecewise
 floor1 representation can overflow the lookup
Also fix a typo in the comments of the original patch.

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

Port r17542 from Tremor.

Reinstate an explicit check against phrasebook dim being set to zero.

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

Port r17539 from Tremor.

Bail out of codebook loading early if the packet doesn't have enough data for
 the size of the codebooks it asked for.

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

Fix incorrect 'ms' argument in ov_time_seek_*() docs.

The actual argument is the time in seconds, as documented, but the
 parameter name 'ms' could lead to confusion.
Thanks to Philipp 'ph3-der-loewe' Schafft <lion@lion.leolix.org> for the
 report.

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

Fix decoder handling of floor0 when the LSP order is 1.

Header setup allows the LSP order to be as low as one, but the code in
 vorbis_lsp_to_curve() assumed it was at least two.
This wasn't terrible in libvorbis... it would multiply a nonsense (but defined)
 value into the output, and nothing more.
In Tremor, it referenced several completely undefined (stack) values, which
 could cause out-of-bounds lookup table accesses and crashes.

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

Make vorbis_synthesis_trackonly error handling match vorbis_synthesis per Mozilla bug487519.patch

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

Updated Visual Studio project files.

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

Added Visual Studiop 2010 project files.

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

Add ov_read_filter.html to documentation files

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

Correct various comment spelling errors.

Based on a patch by Michael Kohler, trac issue #1695.

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

Commit vorbisfile constification patch; affects API but not ABI.
Closes #1593.

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

Fix a chaining bug in raw_seeks where seeking out of the current link
would fail due to not reinitializing the decode machinery.

Also correct a minor and a moderate inefficiency that reduces the
necessary number of seek callback in a search operation by about 2/3.

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

One of the Mozilla fixes for #1573 (working around liboggplay
ignoring an error return value) broke chaining by causing
vorbis_synthesis_init() to fail on a second call.  Fixed.

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

Try out a new seek bisect optimization in vorbisfile.

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

Re-bump to 1.3.1

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

OK, desired floor fit behavior settled

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

reinstantiate floor1 weighting changes.

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

Revert an element of noise norm to earlier behavior for testing.

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

Revert floor1 line fitting to exact 1.2.3 behavior; the apparently better
weighting of 1.3.0pre was slightly dropping PEAQ and slightly increasing
bitrate.

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

Correct arithmetic error in fit weighting

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

Update version string and CHANGES file toward release

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

Simple arithmetic error as part of fix to #1658 caused floor1 fitting to
fall apart in overwhelmingly tonal regions.

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

Leak fix in decode broke codebook load; fixed

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

Some compilers need at least a placeholder here

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

Attempted fix for 32 bit overflow in floor1 line fitting for very low
bitrate modes with oversized blocks.  Should close #1658.

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

We do in fact need automake 1.7; 1.6 will bomb out.

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

Fix leak when aborting out of static_codebook unpack.  Closes #1663.

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

Fix for Mozilla BZ#550184 and Xiph Trac #1656

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

Kill off two harmless gcc warnings in vorbisenc.c

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

Apply patches from Trac #1638, additional application hardening (not
bugfixes, but ignoring easy-to-catch cases of improper lib use) and one
more bitstream guard.

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

22kHz setup was using wrong compand mapping; closes Trac #1635

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

Correct bug in 16kHz coupled books similar to the bug that had popped up in
uncoupled output.  It would cause efficiency loss and unstable tones in
high-rate encoding (q5 and higher).

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

Test 1 through 8 channels, all quality levels, all main sample rates.  This caught another codebook error in the
16kHz books

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