From 99db9321ad29429af345a5677dd24bb1263d21ba Mon Sep 17 00:00:00 2001 From: Ozkan Sezer Date: Mon, 7 Oct 2019 01:04:02 +0300 Subject: [PATCH] external libs (libvorbisidec-1.2.1): updates from xiph.org tremor.git --- .../0000-tremor.git-2e87629f.patch | 102 ++++ .../0000-tremor.git-49c8a74e.patch | 457 ++++++++++++++++++ .../0000-tremor.git-562307a4.patch | 85 ++++ .../0000-tremor.git-7c30a663.patch | 48 ++ .../0000-tremor.git-b56ffce0.patch | 26 + external/libvorbisidec-1.2.1/codebook.c | 18 +- external/libvorbisidec-1.2.1/info.c | 13 +- .../libvorbisidec-1.2.1/iseeking_example.c | 6 +- external/libvorbisidec-1.2.1/ivorbisfile.h | 3 +- external/libvorbisidec-1.2.1/mdct.c | 2 +- external/libvorbisidec-1.2.1/synthesis.c | 4 +- external/libvorbisidec-1.2.1/vorbisfile.c | 232 ++++++--- 12 files changed, 905 insertions(+), 91 deletions(-) create mode 100644 external/libvorbisidec-1.2.1/0000-tremor.git-2e87629f.patch create mode 100644 external/libvorbisidec-1.2.1/0000-tremor.git-49c8a74e.patch create mode 100644 external/libvorbisidec-1.2.1/0000-tremor.git-562307a4.patch create mode 100644 external/libvorbisidec-1.2.1/0000-tremor.git-7c30a663.patch create mode 100644 external/libvorbisidec-1.2.1/0000-tremor.git-b56ffce0.patch diff --git a/external/libvorbisidec-1.2.1/0000-tremor.git-2e87629f.patch b/external/libvorbisidec-1.2.1/0000-tremor.git-2e87629f.patch new file mode 100644 index 00000000..5bc593b7 --- /dev/null +++ b/external/libvorbisidec-1.2.1/0000-tremor.git-2e87629f.patch @@ -0,0 +1,102 @@ +From: Monty +Date: Tue, 24 Jun 2014 14:44:34 +0000 (+0000) +Subject: Commit suggested fix for #2021 Backport additional #1486 -n -r 19164 set keyword... +X-Git-Url: http://git.xiph.org/?p=tremor.git;a=commitdiff_plain;h=2e87629fb56cb23f969fb5862e721fc1a53f9725 + +Commit suggested fix for #2021 Backport additional #1486 -n -r 19164 set keyword property on source files + +git-svn-id: https://svn.xiph.org/trunk/Tremor@19172 0101bb08-14d6-0310-b084-bc0e0c8e3800 +--- + +diff --git a/iseeking_example.c b/iseeking_example.c +index e3bf883..dc971ba 100644 +--- a/iseeking_example.c ++++ b/iseeking_example.c +@@ -12,7 +12,7 @@ + ******************************************************************** + + function: illustrate seeking, and test it too +- last mod: $Id: iseeking_example.c 16037 2009-05-26 21:10:58Z xiphmont $ ++ last mod: $Id$ + + ********************************************************************/ + +diff --git a/mdct.c b/mdct.c +index 4f39e7d..2aed62c 100644 +--- a/mdct.c ++++ b/mdct.c +@@ -13,7 +13,7 @@ + + function: normalized modified discrete cosine transform + power of two length transform only [64 <= n ] +- last mod: $Id: mdct.c,v 1.9 2002/10/16 09:17:39 xiphmont Exp $ ++ last mod: $Id$ + + Original algorithm adapted long ago from _The use of multirate filter + banks for coding of high quality digital audio_, by T. Sporer, +diff --git a/synthesis.c b/synthesis.c +index 38c3eb8..fe63924 100644 +--- a/synthesis.c ++++ b/synthesis.c +@@ -67,7 +67,7 @@ static int _vorbis_synthesis1(vorbis_block *vb,ogg_packet *op,int decodep){ + + /* more setup */ + vb->granulepos=op->granulepos; +- vb->sequence=op->packetno-3; /* first block is third packet */ ++ vb->sequence=op->packetno; /* first block is third packet */ + vb->eofflag=op->e_o_s; + + if(decodep){ +diff --git a/vorbisfile.c b/vorbisfile.c +index 9413289..cd4814d 100644 +--- a/vorbisfile.c ++++ b/vorbisfile.c +@@ -12,7 +12,7 @@ + ******************************************************************** + + function: stdio-based convenience library for opening/seeking/decoding +- last mod: $Id: vorbisfile.c,v 1.6 2003/03/30 23:40:56 xiphmont Exp $ ++ last mod: $Id$ + + ********************************************************************/ + +@@ -1421,8 +1421,21 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ + ogg_int64_t endtime = vf->pcmlengths[link*2+1]+begintime; + ogg_int64_t target=pos-total+begintime; + ogg_int64_t best=-1; ++ int got_page=0; + + ogg_page og; ++ ++ /* if we have only one page, there will be no bisection. Grab the page here */ ++ if(begin==end){ ++ result=_seek_helper(vf,begin); ++ if(result) goto seek_error; ++ ++ result=_get_next_page(vf,&og,1); ++ if(result<0) goto seek_error; ++ ++ got_page=1; ++ } ++ + /* bisection loop */ + while(begindataoffsets[link] && ++ if(got_page && ++ begin == vf->dataoffsets[link] && + ogg_page_serialno(&og)==vf->serialnos[link]){ + + /* Yes, this is the beginning-of-stream case. We already have diff --git a/external/libvorbisidec-1.2.1/0000-tremor.git-49c8a74e.patch b/external/libvorbisidec-1.2.1/0000-tremor.git-49c8a74e.patch new file mode 100644 index 00000000..53c6cb75 --- /dev/null +++ b/external/libvorbisidec-1.2.1/0000-tremor.git-49c8a74e.patch @@ -0,0 +1,457 @@ +From: Monty +Date: Tue, 24 Jun 2014 12:12:08 +0000 (+0000) +Subject: Backport #1486 fixes from libvorbis to Tremor (trunk) +X-Git-Url: http://git.xiph.org/?p=tremor.git;a=commitdiff_plain;h=49c8a74e51a9441e2a8acea65fec7689c807507e + +Backport #1486 fixes from libvorbis to Tremor (trunk) + +git-svn-id: https://svn.xiph.org/trunk/Tremor@19170 0101bb08-14d6-0310-b084-bc0e0c8e3800 +--- + +diff --git a/iseeking_example.c b/iseeking_example.c +index de534eb..e3bf883 100644 +--- a/iseeking_example.c ++++ b/iseeking_example.c +@@ -160,7 +160,7 @@ int main(){ + (long)pcmlength); + + for(i=0;i<1000;i++){ +- ogg_int64_t val=(double)rand()*pcmlength/RAND_MAX; ++ ogg_int64_t val=i==0?0:(double)rand()*pcmlength/RAND_MAX; + fprintf(stderr,"\r\t%d [pcm position %ld]... ",i,(long)val); + ret=ov_pcm_seek_page(&ov,val); + if(ret<0){ +@@ -179,7 +179,7 @@ int main(){ + (long)pcmlength); + + for(i=0;i<1000;i++){ +- ogg_int64_t val=(double)rand()*pcmlength/RAND_MAX; ++ ogg_int64_t val=i==0?0:(double)rand()*pcmlength/RAND_MAX; + fprintf(stderr,"\r\t%d [pcm position %ld]... ",i,(long)val); + ret=ov_pcm_seek(&ov,val); + if(ret<0){ +diff --git a/ivorbisfile.h b/ivorbisfile.h +index fda6205..f6ecb0e 100644 +--- a/ivorbisfile.h ++++ b/ivorbisfile.h +@@ -26,7 +26,8 @@ extern "C" + #include + #include "ivorbiscodec.h" + +-#define CHUNKSIZE 1024 ++#define CHUNKSIZE 65535 ++#define READSIZE 1024 + /* The function prototypes for the callbacks are basically the same as for + * the stdio functions fread, fseek, fclose, ftell. + * The one difference is that the FILE * arguments have been replaced with +diff --git a/vorbisfile.c b/vorbisfile.c +index ac0eb88..9413289 100644 +--- a/vorbisfile.c ++++ b/vorbisfile.c +@@ -6,7 +6,7 @@ + * GOVERNED BY A BSD-STYLE SOURCE LICENSE INCLUDED WITH THIS SOURCE * + * IN 'COPYING'. PLEASE READ THESE TERMS BEFORE DISTRIBUTING. * + * * +- * THE OggVorbis 'TREMOR' SOURCE CODE IS (C) COPYRIGHT 1994-2009 * ++ * THE OggVorbis 'TREMOR' SOURCE CODE IS (C) COPYRIGHT 1994-2014 * + * BY THE Xiph.Org FOUNDATION http://www.xiph.org/ * + * * + ******************************************************************** +@@ -63,8 +63,8 @@ static long _get_data(OggVorbis_File *vf){ + errno=0; + if(!(vf->callbacks.read_func))return(-1); + if(vf->datasource){ +- char *buffer=ogg_sync_buffer(&vf->oy,CHUNKSIZE); +- long bytes=(vf->callbacks.read_func)(buffer,1,CHUNKSIZE,vf->datasource); ++ char *buffer=ogg_sync_buffer(&vf->oy,READSIZE); ++ long bytes=(vf->callbacks.read_func)(buffer,1,READSIZE,vf->datasource); + if(bytes>0)ogg_sync_wrote(&vf->oy,bytes); + if(bytes==0 && errno)return(-1); + return(bytes); +@@ -75,11 +75,14 @@ static long _get_data(OggVorbis_File *vf){ + /* save a tiny smidge of verbosity to make the code more readable */ + static int _seek_helper(OggVorbis_File *vf,ogg_int64_t offset){ + if(vf->datasource){ +- if(!(vf->callbacks.seek_func)|| +- (vf->callbacks.seek_func)(vf->datasource, offset, SEEK_SET) == -1) +- return OV_EREAD; +- vf->offset=offset; +- ogg_sync_reset(&vf->oy); ++ /* only seek if the file position isn't already there */ ++ if(vf->offset != offset){ ++ if(!(vf->callbacks.seek_func)|| ++ (vf->callbacks.seek_func)(vf->datasource, offset, SEEK_SET) == -1) ++ return OV_EREAD; ++ vf->offset=offset; ++ ogg_sync_reset(&vf->oy); ++ } + }else{ + /* shouldn't happen unless someone writes a broken callback */ + return OV_EFAULT; +@@ -133,14 +136,12 @@ static ogg_int64_t _get_next_page(OggVorbis_File *vf,ogg_page *og, + } + } + +-/* find the latest page beginning before the current stream cursor +- position. Much dirtier than the above as Ogg doesn't have any +- backward search linkage. no 'readp' as it will certainly have to +- read. */ ++/* find the latest page beginning before the passed in position. Much ++ dirtier than the above as Ogg doesn't have any backward search ++ linkage. no 'readp' as it will certainly have to read. */ + /* returns offset or OV_EREAD, OV_FAULT */ +-static ogg_int64_t _get_prev_page(OggVorbis_File *vf,ogg_page *og){ +- ogg_int64_t begin=vf->offset; +- ogg_int64_t end=begin; ++static ogg_int64_t _get_prev_page(OggVorbis_File *vf,ogg_int64_t begin,ogg_page *og){ ++ ogg_int64_t end = begin; + ogg_int64_t ret; + ogg_int64_t offset=-1; + +@@ -215,11 +216,10 @@ static int _lookup_page_serialno(ogg_page *og, ogg_uint32_t *serialno_list, int + info of last page of the matching serial number instead of the very + last page. If no page of the specified serialno is seen, it will + return the info of last page and alter *serialno. */ +-static ogg_int64_t _get_prev_page_serial(OggVorbis_File *vf, ++static ogg_int64_t _get_prev_page_serial(OggVorbis_File *vf, ogg_int64_t begin, + ogg_uint32_t *serial_list, int serial_n, + int *serialno, ogg_int64_t *granpos){ + ogg_page og; +- ogg_int64_t begin=vf->offset; + ogg_int64_t end=begin; + ogg_int64_t ret; + +@@ -488,10 +488,10 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, + down to (or just started with) a single link. Now we need to + find the last vorbis page belonging to the first vorbis stream + for this link. */ +- ++ searched = end; + while(endserial != serialno){ + endserial = serialno; +- vf->offset=_get_prev_page_serial(vf,currentno_list,currentnos,&endserial,&endgran); ++ searched=_get_prev_page_serial(vf,searched,currentno_list,currentnos,&endserial,&endgran); + } + + vf->links=m+1; +@@ -508,14 +508,19 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, + + vf->offsets[m+1]=end; + vf->offsets[m]=begin; +- vf->pcmlengths[m*2+1]=endgran; ++ vf->pcmlengths[m*2+1]=(endgran<0?0:endgran); + + }else{ + ++ /* last page is not in the starting stream's serial number list, ++ so we have multiple links. Find where the stream that begins ++ our bisection ends. */ ++ + ogg_uint32_t *next_serialno_list=NULL; + int next_serialnos=0; + vorbis_info vi; + vorbis_comment vc; ++ int testserial = serialno+1; + + /* the below guards against garbage seperating the last and + first pages of two links. */ +@@ -528,10 +533,8 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, + bisect=(searched+endsearched)/2; + } + +- if(bisect != vf->offset){ +- ret=_seek_helper(vf,bisect); +- if(ret)return(ret); +- } ++ ret=_seek_helper(vf,bisect); ++ if(ret)return(ret); + + last=_get_next_page(vf,&og,-1); + if(last==OV_EREAD)return(OV_EREAD); +@@ -544,28 +547,22 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, + } + + /* Bisection point found */ +- + /* for the time being, fetch end PCM offset the simple way */ +- { +- int testserial = serialno+1; +- vf->offset = next; +- while(testserial != serialno){ +- testserial = serialno; +- vf->offset=_get_prev_page_serial(vf,currentno_list,currentnos,&testserial,&searchgran); +- } ++ searched = next; ++ while(testserial != serialno){ ++ testserial = serialno; ++ searched = _get_prev_page_serial(vf,searched,currentno_list,currentnos,&testserial,&searchgran); + } + +- if(vf->offset!=next){ +- ret=_seek_helper(vf,next); +- if(ret)return(ret); +- } ++ ret=_seek_helper(vf,next); ++ if(ret)return(ret); + + ret=_fetch_headers(vf,&vi,&vc,&next_serialno_list,&next_serialnos,NULL); + if(ret)return(ret); + serialno = vf->os.serialno; + dataoffset = vf->offset; + +- /* this will consume a page, however the next bistection always ++ /* this will consume a page, however the next bisection always + starts with a raw seek */ + pcmoffset = _initial_pcmoffset(vf,&vi); + +@@ -585,6 +582,7 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, + vf->pcmlengths[m*2+1]=searchgran; + vf->pcmlengths[m*2+2]=pcmoffset; + vf->pcmlengths[m*2+3]-=pcmoffset; ++ if(vf->pcmlengths[m*2+3]<0)vf->pcmlengths[m*2+3]=0; + + } + return(0); +@@ -632,11 +630,11 @@ static int _open_seekable2(OggVorbis_File *vf){ + /* Get the offset of the last page of the physical bitstream, or, if + we're lucky the last vorbis page of this link as most OggVorbis + files will contain a single logical bitstream */ +- end=_get_prev_page_serial(vf,vf->serialnos+2,vf->serialnos[1],&endserial,&endgran); ++ end=_get_prev_page_serial(vf,vf->end,vf->serialnos+2,vf->serialnos[1],&endserial,&endgran); + if(end<0)return(end); + + /* now determine bitstream structure recursively */ +- if(_bisect_forward_serialno(vf,0,dataoffset,vf->offset,endgran,endserial, ++ if(_bisect_forward_serialno(vf,0,dataoffset,end,endgran,endserial, + vf->serialnos+2,vf->serialnos[1],0)<0)return(OV_EREAD); + + vf->offsets[0]=0; +@@ -644,6 +642,7 @@ static int _open_seekable2(OggVorbis_File *vf){ + vf->dataoffsets[0]=dataoffset; + vf->pcmlengths[0]=pcmoffset; + vf->pcmlengths[1]-=pcmoffset; ++ if(vf->pcmlengths[1]<0)vf->pcmlengths[1]=0; + + return(ov_raw_seek(vf,dataoffset)); + } +@@ -711,7 +710,7 @@ static int _fetch_and_process_packet(OggVorbis_File *vf, + if(oldsamples)return(OV_EFAULT); + + vorbis_synthesis_blockin(&vf->vd,&vf->vb); +- vf->samptrack+=vorbis_synthesis_pcmout(&vf->vd,NULL)-oldsamples; ++ vf->samptrack+=vorbis_synthesis_pcmout(&vf->vd,NULL); + vf->bittrack+=op_ptr->bytes*8; + } + +@@ -1403,22 +1402,28 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ + if(pos>=total)break; + } + +- /* search within the logical bitstream for the page with the highest +- pcm_pos preceding (or equal to) pos. There is a danger here; +- missing pages or incorrect frame number information in the +- bitstream could make our task impossible. Account for that (it +- would be an error condition) */ ++ /* Search within the logical bitstream for the page with the highest ++ pcm_pos preceding pos. If we're looking for a position on the ++ first page, bisection will halt without finding our position as ++ it's before the first explicit granulepos fencepost. That case is ++ handled separately below. ++ ++ There is a danger here; missing pages or incorrect frame number ++ information in the bitstream could make our task impossible. ++ Account for that (it would be an error condition) */ ++ ++ /* new search algorithm originally by HB (Nicholas Vinen) */ + +- /* new search algorithm by HB (Nicholas Vinen) */ + { + ogg_int64_t end=vf->offsets[link+1]; +- ogg_int64_t begin=vf->offsets[link]; ++ ogg_int64_t begin=vf->dataoffsets[link]; + ogg_int64_t begintime = vf->pcmlengths[link*2]; + ogg_int64_t endtime = vf->pcmlengths[link*2+1]+begintime; + ogg_int64_t target=pos-total+begintime; +- ogg_int64_t best=begin; ++ ogg_int64_t best=-1; + + ogg_page og; ++ /* bisection loop */ + while(beginoffset){ +- result=_seek_helper(vf,bisect); +- if(result) goto seek_error; +- } ++ result=_seek_helper(vf,bisect); ++ if(result) goto seek_error; + ++ /* read loop within the bisection loop */ + while(beginoffset); + if(result==OV_EREAD) goto seek_error; + if(result<0){ ++ /* there is no next page! */ + if(bisect<=begin+1) +- end=begin; /* found it */ ++ /* No bisection left to perform. We've either found the ++ best candidate already or failed. Exit loop. */ ++ end=begin; + else{ ++ /* We tried to load a fraction of the last page; back up a ++ bit and try to get the whole last page */ + if(bisect==0) goto seek_error; + bisect-=CHUNKSIZE; ++ ++ /* don't repeat/loop on a read we've already performed */ + if(bisect<=begin)bisect=begin+1; ++ ++ /* seek and continue bisection */ + result=_seek_helper(vf,bisect); + if(result) goto seek_error; + } + }else{ + ogg_int64_t granulepos; + ++ /* got a page. analyze it */ ++ /* only consider pages from primary vorbis stream */ + if(ogg_page_serialno(&og)!=vf->serialnos[link]) + continue; + ++ /* only consider pages with the granulepos set */ + granulepos=ogg_page_granulepos(&og); + if(granulepos==-1)continue; + + if(granuleposoffset; /* raw offset of next page */ + begintime=granulepos; + ++ /* if we're before our target but within a short distance, ++ don't bisect; read forward */ + if(target-begintime>44100)break; +- bisect=begin; /* *not* begin + 1 */ ++ ++ bisect=begin; /* *not* begin + 1 as above */ + }else{ +- if(bisect<=begin+1) +- end=begin; /* found it */ +- else{ +- if(end==vf->offset){ /* we're pretty close - we'd be stuck in */ ++ ++ /* This is one of our pages, but the granpos is ++ post-target; it is not a bisection return ++ candidate. (The only way we'd use it is if it's the ++ first page in the stream; we handle that case later ++ outside the bisection) */ ++ if(bisect<=begin+1){ ++ /* No bisection left to perform. We've either found the ++ best candidate already or failed. Exit loop. */ ++ end=begin; ++ }else{ ++ if(end==vf->offset){ ++ /* bisection read to the end; use the known page ++ boundary (result) to update bisection, back up a ++ little bit, and try again */ + end=result; +- bisect-=CHUNKSIZE; /* an endless loop otherwise. */ ++ bisect-=CHUNKSIZE; + if(bisect<=begin)bisect=begin+1; + result=_seek_helper(vf,bisect); + if(result) goto seek_error; + }else{ ++ /* Normal bisection */ + end=bisect; + endtime=granulepos; + break; +@@ -1488,9 +1521,45 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ + } + } + +- /* found our page. seek to it, update pcm offset. Easier case than +- raw_seek, don't keep packets preceding granulepos. */ +- { ++ /* Out of bisection: did it 'fail?' */ ++ if(best == -1){ ++ ++ /* Check the 'looking for data in first page' special case; ++ bisection would 'fail' because our search target was before the ++ first PCM granule position fencepost. */ ++ ++ if(begin == vf->dataoffsets[link] && ++ ogg_page_serialno(&og)==vf->serialnos[link]){ ++ ++ /* Yes, this is the beginning-of-stream case. We already have ++ our page, right at the beginning of PCM data. Set state ++ and return. */ ++ ++ vf->pcm_offset=total; ++ ++ if(link!=vf->current_link){ ++ /* Different link; dump entire decode machine */ ++ _decode_clear(vf); ++ ++ vf->current_link=link; ++ vf->current_serialno=vf->serialnos[link]; ++ vf->ready_state=STREAMSET; ++ ++ }else{ ++ vorbis_synthesis_restart(&vf->vd); ++ } ++ ++ ogg_stream_reset_serialno(&vf->os,vf->current_serialno); ++ ogg_stream_pagein(&vf->os,&og); ++ ++ }else ++ goto seek_error; ++ ++ }else{ ++ ++ /* Bisection found our page. seek to it, update pcm offset. Easier case than ++ raw_seek, don't keep packets preceding granulepos. */ ++ + ogg_page og; + ogg_packet op; + +@@ -1520,23 +1589,23 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ + while(1){ + result=ogg_stream_packetpeek(&vf->os,&op); + if(result==0){ +- /* !!! the packet finishing this page originated on a +- preceding page. Keep fetching previous pages until we +- get one with a granulepos or without the 'continued' flag +- set. Then just use raw_seek for simplicity. */ +- +- result=_seek_helper(vf,best); +- if(result<0) goto seek_error; +- +- while(1){ +- result=_get_prev_page(vf,&og); ++ /* No packet returned; we exited the bisection with 'best' ++ pointing to a page with a granule position, so the packet ++ finishing this page ('best') originated on a preceding ++ page. Keep fetching previous pages until we get one with ++ a granulepos or without the 'continued' flag set. Then ++ just use raw_seek for simplicity. */ ++ /* Do not rewind past the beginning of link data; if we do, ++ it's either a bug or a broken stream */ ++ result=best; ++ while(result>vf->dataoffsets[link]){ ++ result=_get_prev_page(vf,result,&og); + if(result<0) goto seek_error; + if(ogg_page_serialno(&og)==vf->current_serialno && + (ogg_page_granulepos(&og)>-1 || + !ogg_page_continued(&og))){ + return ov_raw_seek(vf,result); + } +- vf->offset=result; + } + } + if(result<0){ diff --git a/external/libvorbisidec-1.2.1/0000-tremor.git-562307a4.patch b/external/libvorbisidec-1.2.1/0000-tremor.git-562307a4.patch new file mode 100644 index 00000000..1abced42 --- /dev/null +++ b/external/libvorbisidec-1.2.1/0000-tremor.git-562307a4.patch @@ -0,0 +1,85 @@ +From: Thomas Daede +Date: Fri, 16 Mar 2018 04:47:09 +0000 (-0700) +Subject: Prevent out-of-bounds write in codebook decoding. +X-Git-Url: http://git.xiph.org/?p=tremor.git;a=commitdiff_plain;h=562307a4a7082e24553f3d2c55dab397a17c4b4f + +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. +--- + +diff --git a/codebook.c b/codebook.c +index f8b7983..1e1ae8a 100644 +--- a/codebook.c ++++ b/codebook.c +@@ -258,7 +258,7 @@ long vorbis_book_decodevs_add(codebook *book,ogg_int32_t *a, + t[i] = book->valuelist+entry[i]*book->dim; + } + for(i=0,o=0;idim;i++,o+=step) +- for (j=0;j>shift; + }else{ + for (i = 0; i < step; i++) { +@@ -267,7 +267,7 @@ long vorbis_book_decodevs_add(codebook *book,ogg_int32_t *a, + t[i] = book->valuelist+entry[i]*book->dim; + } + for(i=0,o=0;idim;i++,o+=step) +- for (j=0;jvaluelist+entry*book->dim; +- for (j=0;jdim;) ++ for (j=0;idim;) + a[i++]+=t[j++]>>shift; + } + }else{ +@@ -295,7 +295,7 @@ long vorbis_book_decodev_add(codebook *book,ogg_int32_t *a, + entry = decode_packed_entry_number(book,b); + if(entry==-1)return(-1); + t = book->valuelist+entry*book->dim; +- for (j=0;jdim;) ++ for (j=0;idim;) + a[i++]+=t[j++]<<-shift; + } + } +@@ -352,15 +352,15 @@ long vorbis_book_decodevv_add(codebook *book,ogg_int32_t **a,\ + long i,j,entry; + int chptr=0; + int shift=point-book->binarypoint; +- ++ int m=offset+n; + if(shift>=0){ + +- for(i=offset;ivaluelist+entry*book->dim; +- for (j=0;jdim;j++){ ++ for (j=0;idim;j++){ + a[chptr++][i]+=t[j]>>shift; + if(chptr==ch){ + chptr=0; +@@ -371,12 +371,12 @@ long vorbis_book_decodevv_add(codebook *book,ogg_int32_t **a,\ + } + }else{ + +- for(i=offset;ivaluelist+entry*book->dim; +- for (j=0;jdim;j++){ ++ for (j=0;idim;j++){ + a[chptr++][i]+=t[j]<<-shift; + if(chptr==ch){ + chptr=0; diff --git a/external/libvorbisidec-1.2.1/0000-tremor.git-7c30a663.patch b/external/libvorbisidec-1.2.1/0000-tremor.git-7c30a663.patch new file mode 100644 index 00000000..7a833f22 --- /dev/null +++ b/external/libvorbisidec-1.2.1/0000-tremor.git-7c30a663.patch @@ -0,0 +1,48 @@ +From: Tim Terriberry +Date: Mon, 19 Mar 2018 20:08:21 +0000 (-0700) +Subject: Port r19426 from libvorbis. +X-Git-Url: http://git.xiph.org/?p=tremor.git;a=commitdiff_plain;h=7c30a66346199f3f09017a09567c6c8a3a0eedc8 + +Port r19426 from libvorbis. + +Reject multiple headers of the same type. +--- + +diff --git a/info.c b/info.c +index 3f3308e..24e24ac 100644 +--- a/info.c ++++ b/info.c +@@ -227,7 +227,6 @@ static int _vorbis_unpack_comment(vorbis_comment *vc,oggpack_buffer *opb){ + static int _vorbis_unpack_books(vorbis_info *vi,oggpack_buffer *opb){ + codec_setup_info *ci=(codec_setup_info *)vi->codec_setup; + int i; +- if(!ci)return(OV_EFAULT); + + /* codebooks */ + ci->books=oggpack_read(opb,8)+1; +@@ -367,6 +366,10 @@ int vorbis_synthesis_headerin(vorbis_info *vi,vorbis_comment *vc,ogg_packet *op) + /* um... we didn't get the initial header */ + return(OV_EBADHEADER); + } ++ if(vc->vendor!=NULL){ ++ /* previously initialized comment header */ ++ return(OV_EBADHEADER); ++ } + + return(_vorbis_unpack_comment(vc,&opb)); + +@@ -375,6 +378,14 @@ int vorbis_synthesis_headerin(vorbis_info *vi,vorbis_comment *vc,ogg_packet *op) + /* um... we didn;t get the initial header or comments yet */ + return(OV_EBADHEADER); + } ++ if(vi->codec_setup==NULL){ ++ /* improperly initialized vorbis_info */ ++ return(OV_EFAULT); ++ } ++ if(((codec_setup_info *)vi->codec_setup)->books>0){ ++ /* previously initialized setup header */ ++ return(OV_EBADHEADER); ++ } + + return(_vorbis_unpack_books(vi,&opb)); + diff --git a/external/libvorbisidec-1.2.1/0000-tremor.git-b56ffce0.patch b/external/libvorbisidec-1.2.1/0000-tremor.git-b56ffce0.patch new file mode 100644 index 00000000..7ca4f8b3 --- /dev/null +++ b/external/libvorbisidec-1.2.1/0000-tremor.git-b56ffce0.patch @@ -0,0 +1,26 @@ +From: Tim Terriberry +Date: Sun, 4 Jan 2015 19:11:49 +0000 (+0000) +Subject: Port r19419 to Tremor. +X-Git-Url: http://git.xiph.org/?p=tremor.git;a=commitdiff_plain;h=b56ffce0c0773ec5ca04c466bc00b1bbcaf65aef + +Port r19419 to Tremor. + +Fix crash on corrupt input file (invalid mode index) + + +git-svn-id: https://svn.xiph.org/trunk/Tremor@19420 0101bb08-14d6-0310-b084-bc0e0c8e3800 +--- + +diff --git a/synthesis.c b/synthesis.c +index fe63924..d22cb82 100644 +--- a/synthesis.c ++++ b/synthesis.c +@@ -124,7 +124,7 @@ long vorbis_packet_blocksize(vorbis_info *vi,ogg_packet *op){ + /* read our mode and pre/post windowsize */ + mode=oggpack_read(&opb,modebits); + } +- if(mode==-1)return(OV_EBADPACKET); ++ if(mode==-1 || !ci->mode_param[mode])return(OV_EBADPACKET); + return(ci->blocksizes[ci->mode_param[mode]->blockflag]); + } + diff --git a/external/libvorbisidec-1.2.1/codebook.c b/external/libvorbisidec-1.2.1/codebook.c index 21dc1d46..1bbba8f1 100644 --- a/external/libvorbisidec-1.2.1/codebook.c +++ b/external/libvorbisidec-1.2.1/codebook.c @@ -258,7 +258,7 @@ long vorbis_book_decodevs_add(codebook *book,ogg_int32_t *a, t[i] = book->valuelist+entry[i]*book->dim; } for(i=0,o=0;idim;i++,o+=step) - for (j=0;j>shift; }else{ for (i = 0; i < step; i++) { @@ -267,7 +267,7 @@ long vorbis_book_decodevs_add(codebook *book,ogg_int32_t *a, t[i] = book->valuelist+entry[i]*book->dim; } for(i=0,o=0;idim;i++,o+=step) - for (j=0;jvaluelist+entry*book->dim; - for (j=0;jdim;) + for (j=0;idim;) a[i++]+=t[j++]>>shift; } }else{ @@ -295,7 +295,7 @@ long vorbis_book_decodev_add(codebook *book,ogg_int32_t *a, entry = decode_packed_entry_number(book,b); if(entry==-1)return(-1); t = book->valuelist+entry*book->dim; - for (j=0;jdim;) + for (j=0;idim;) a[i++]+=t[j++]<<-shift; } } @@ -352,15 +352,15 @@ long vorbis_book_decodevv_add(codebook *book,ogg_int32_t **a,\ long i,j,entry; int chptr=0; int shift=point-book->binarypoint; - + int m=offset+n; if(shift>=0){ - for(i=offset;ivaluelist+entry*book->dim; - for (j=0;jdim;j++){ + for (j=0;idim;j++){ a[chptr++][i]+=t[j]>>shift; if(chptr==ch){ chptr=0; @@ -371,12 +371,12 @@ long vorbis_book_decodevv_add(codebook *book,ogg_int32_t **a,\ } }else{ - for(i=offset;ivaluelist+entry*book->dim; - for (j=0;jdim;j++){ + for (j=0;idim;j++){ a[chptr++][i]+=t[j]<<-shift; if(chptr==ch){ chptr=0; diff --git a/external/libvorbisidec-1.2.1/info.c b/external/libvorbisidec-1.2.1/info.c index efa9b286..dd5b7ad1 100644 --- a/external/libvorbisidec-1.2.1/info.c +++ b/external/libvorbisidec-1.2.1/info.c @@ -230,7 +230,6 @@ static int _vorbis_unpack_comment(vorbis_comment *vc,oggpack_buffer *opb){ static int _vorbis_unpack_books(vorbis_info *vi,oggpack_buffer *opb){ codec_setup_info *ci=(codec_setup_info *)vi->codec_setup; int i; - if(!ci)return(OV_EFAULT); /* codebooks */ ci->books=oggpack_read(opb,8)+1; @@ -370,6 +369,10 @@ int vorbis_synthesis_headerin(vorbis_info *vi,vorbis_comment *vc,ogg_packet *op) /* um... we didn't get the initial header */ return(OV_EBADHEADER); } + if(vc->vendor!=NULL){ + /* previously initialized comment header */ + return(OV_EBADHEADER); + } return(_vorbis_unpack_comment(vc,&opb)); @@ -378,6 +381,14 @@ int vorbis_synthesis_headerin(vorbis_info *vi,vorbis_comment *vc,ogg_packet *op) /* um... we didn;t get the initial header or comments yet */ return(OV_EBADHEADER); } + if(vi->codec_setup==NULL){ + /* improperly initialized vorbis_info */ + return(OV_EFAULT); + } + if(((codec_setup_info *)vi->codec_setup)->books>0){ + /* previously initialized setup header */ + return(OV_EBADHEADER); + } return(_vorbis_unpack_books(vi,&opb)); diff --git a/external/libvorbisidec-1.2.1/iseeking_example.c b/external/libvorbisidec-1.2.1/iseeking_example.c index de534eba..dc971ba4 100644 --- a/external/libvorbisidec-1.2.1/iseeking_example.c +++ b/external/libvorbisidec-1.2.1/iseeking_example.c @@ -12,7 +12,7 @@ ******************************************************************** function: illustrate seeking, and test it too - last mod: $Id: iseeking_example.c 16037 2009-05-26 21:10:58Z xiphmont $ + last mod: $Id$ ********************************************************************/ @@ -160,7 +160,7 @@ int main(){ (long)pcmlength); for(i=0;i<1000;i++){ - ogg_int64_t val=(double)rand()*pcmlength/RAND_MAX; + ogg_int64_t val=i==0?0:(double)rand()*pcmlength/RAND_MAX; fprintf(stderr,"\r\t%d [pcm position %ld]... ",i,(long)val); ret=ov_pcm_seek_page(&ov,val); if(ret<0){ @@ -179,7 +179,7 @@ int main(){ (long)pcmlength); for(i=0;i<1000;i++){ - ogg_int64_t val=(double)rand()*pcmlength/RAND_MAX; + ogg_int64_t val=i==0?0:(double)rand()*pcmlength/RAND_MAX; fprintf(stderr,"\r\t%d [pcm position %ld]... ",i,(long)val); ret=ov_pcm_seek(&ov,val); if(ret<0){ diff --git a/external/libvorbisidec-1.2.1/ivorbisfile.h b/external/libvorbisidec-1.2.1/ivorbisfile.h index fda62056..f6ecb0e4 100644 --- a/external/libvorbisidec-1.2.1/ivorbisfile.h +++ b/external/libvorbisidec-1.2.1/ivorbisfile.h @@ -26,7 +26,8 @@ extern "C" #include #include "ivorbiscodec.h" -#define CHUNKSIZE 1024 +#define CHUNKSIZE 65535 +#define READSIZE 1024 /* The function prototypes for the callbacks are basically the same as for * the stdio functions fread, fseek, fclose, ftell. * The one difference is that the FILE * arguments have been replaced with diff --git a/external/libvorbisidec-1.2.1/mdct.c b/external/libvorbisidec-1.2.1/mdct.c index a0a5756a..3ad4483f 100644 --- a/external/libvorbisidec-1.2.1/mdct.c +++ b/external/libvorbisidec-1.2.1/mdct.c @@ -13,7 +13,7 @@ function: normalized modified discrete cosine transform power of two length transform only [64 <= n ] - last mod: $Id: mdct.c,v 1.9 2002/10/16 09:17:39 xiphmont Exp $ + last mod: $Id$ Original algorithm adapted long ago from _The use of multirate filter banks for coding of high quality digital audio_, by T. Sporer, diff --git a/external/libvorbisidec-1.2.1/synthesis.c b/external/libvorbisidec-1.2.1/synthesis.c index 38c3eb8c..d22cb823 100644 --- a/external/libvorbisidec-1.2.1/synthesis.c +++ b/external/libvorbisidec-1.2.1/synthesis.c @@ -67,7 +67,7 @@ static int _vorbis_synthesis1(vorbis_block *vb,ogg_packet *op,int decodep){ /* more setup */ vb->granulepos=op->granulepos; - vb->sequence=op->packetno-3; /* first block is third packet */ + vb->sequence=op->packetno; /* first block is third packet */ vb->eofflag=op->e_o_s; if(decodep){ @@ -124,7 +124,7 @@ long vorbis_packet_blocksize(vorbis_info *vi,ogg_packet *op){ /* read our mode and pre/post windowsize */ mode=oggpack_read(&opb,modebits); } - if(mode==-1)return(OV_EBADPACKET); + if(mode==-1 || !ci->mode_param[mode])return(OV_EBADPACKET); return(ci->blocksizes[ci->mode_param[mode]->blockflag]); } diff --git a/external/libvorbisidec-1.2.1/vorbisfile.c b/external/libvorbisidec-1.2.1/vorbisfile.c index 1706fa37..afb4f054 100644 --- a/external/libvorbisidec-1.2.1/vorbisfile.c +++ b/external/libvorbisidec-1.2.1/vorbisfile.c @@ -6,13 +6,13 @@ * GOVERNED BY A BSD-STYLE SOURCE LICENSE INCLUDED WITH THIS SOURCE * * IN 'COPYING'. PLEASE READ THESE TERMS BEFORE DISTRIBUTING. * * * - * THE OggVorbis 'TREMOR' SOURCE CODE IS (C) COPYRIGHT 1994-2009 * + * THE OggVorbis 'TREMOR' SOURCE CODE IS (C) COPYRIGHT 1994-2014 * * BY THE Xiph.Org FOUNDATION http://www.xiph.org/ * * * ******************************************************************** function: stdio-based convenience library for opening/seeking/decoding - last mod: $Id: vorbisfile.c,v 1.6 2003/03/30 23:40:56 xiphmont Exp $ + last mod: $Id$ ********************************************************************/ @@ -63,8 +63,8 @@ static long _get_data(OggVorbis_File *vf){ errno=0; if(!(vf->callbacks.read_func))return(-1); if(vf->datasource){ - char *buffer=ogg_sync_buffer(&vf->oy,CHUNKSIZE); - long bytes=(vf->callbacks.read_func)(buffer,1,CHUNKSIZE,vf->datasource); + char *buffer=ogg_sync_buffer(&vf->oy,READSIZE); + long bytes=(vf->callbacks.read_func)(buffer,1,READSIZE,vf->datasource); if(bytes>0)ogg_sync_wrote(&vf->oy,bytes); if(bytes==0 && errno)return(-1); return(bytes); @@ -75,11 +75,14 @@ static long _get_data(OggVorbis_File *vf){ /* save a tiny smidge of verbosity to make the code more readable */ static int _seek_helper(OggVorbis_File *vf,ogg_int64_t offset){ if(vf->datasource){ - if(!(vf->callbacks.seek_func)|| - (vf->callbacks.seek_func)(vf->datasource, offset, SEEK_SET) == -1) - return OV_EREAD; - vf->offset=offset; - ogg_sync_reset(&vf->oy); + /* only seek if the file position isn't already there */ + if(vf->offset != offset){ + if(!(vf->callbacks.seek_func)|| + (vf->callbacks.seek_func)(vf->datasource, offset, SEEK_SET) == -1) + return OV_EREAD; + vf->offset=offset; + ogg_sync_reset(&vf->oy); + } }else{ /* shouldn't happen unless someone writes a broken callback */ return OV_EFAULT; @@ -133,14 +136,12 @@ static ogg_int64_t _get_next_page(OggVorbis_File *vf,ogg_page *og, } } -/* find the latest page beginning before the current stream cursor - position. Much dirtier than the above as Ogg doesn't have any - backward search linkage. no 'readp' as it will certainly have to - read. */ +/* find the latest page beginning before the passed in position. Much + dirtier than the above as Ogg doesn't have any backward search + linkage. no 'readp' as it will certainly have to read. */ /* returns offset or OV_EREAD, OV_FAULT */ -static ogg_int64_t _get_prev_page(OggVorbis_File *vf,ogg_page *og){ - ogg_int64_t begin=vf->offset; - ogg_int64_t end=begin; +static ogg_int64_t _get_prev_page(OggVorbis_File *vf,ogg_int64_t begin,ogg_page *og){ + ogg_int64_t end = begin; ogg_int64_t ret; ogg_int64_t offset=-1; @@ -215,11 +216,10 @@ static int _lookup_page_serialno(ogg_page *og, ogg_uint32_t *serialno_list, int info of last page of the matching serial number instead of the very last page. If no page of the specified serialno is seen, it will return the info of last page and alter *serialno. */ -static ogg_int64_t _get_prev_page_serial(OggVorbis_File *vf, +static ogg_int64_t _get_prev_page_serial(OggVorbis_File *vf, ogg_int64_t begin, ogg_uint32_t *serial_list, int serial_n, int *serialno, ogg_int64_t *granpos){ ogg_page og; - ogg_int64_t begin=vf->offset; ogg_int64_t end=begin; ogg_int64_t ret; @@ -490,10 +490,10 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, down to (or just started with) a single link. Now we need to find the last vorbis page belonging to the first vorbis stream for this link. */ - + searched = end; while(endserial != serialno){ endserial = serialno; - vf->offset=_get_prev_page_serial(vf,currentno_list,currentnos,&endserial,&endgran); + searched=_get_prev_page_serial(vf,searched,currentno_list,currentnos,&endserial,&endgran); } vf->links=m+1; @@ -510,14 +510,19 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, vf->offsets[m+1]=end; vf->offsets[m]=begin; - vf->pcmlengths[m*2+1]=endgran; + vf->pcmlengths[m*2+1]=(endgran<0?0:endgran); }else{ + /* last page is not in the starting stream's serial number list, + so we have multiple links. Find where the stream that begins + our bisection ends. */ + ogg_uint32_t *next_serialno_list=NULL; int next_serialnos=0; vorbis_info vi; vorbis_comment vc; + int testserial = serialno+1; /* the below guards against garbage seperating the last and first pages of two links. */ @@ -530,10 +535,8 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, bisect=(searched+endsearched)/2; } - if(bisect != vf->offset){ - ret=_seek_helper(vf,bisect); - if(ret)return(ret); - } + ret=_seek_helper(vf,bisect); + if(ret)return(ret); last=_get_next_page(vf,&og,-1); if(last==OV_EREAD)return(OV_EREAD); @@ -546,28 +549,22 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, } /* Bisection point found */ - /* for the time being, fetch end PCM offset the simple way */ - { - int testserial = serialno+1; - vf->offset = next; - while(testserial != serialno){ - testserial = serialno; - vf->offset=_get_prev_page_serial(vf,currentno_list,currentnos,&testserial,&searchgran); - } + searched = next; + while(testserial != serialno){ + testserial = serialno; + searched = _get_prev_page_serial(vf,searched,currentno_list,currentnos,&testserial,&searchgran); } - if(vf->offset!=next){ - ret=_seek_helper(vf,next); - if(ret)return(ret); - } + ret=_seek_helper(vf,next); + if(ret)return(ret); ret=_fetch_headers(vf,&vi,&vc,&next_serialno_list,&next_serialnos,NULL); if(ret)return(ret); serialno = vf->os.serialno; dataoffset = vf->offset; - /* this will consume a page, however the next bistection always + /* this will consume a page, however the next bisection always starts with a raw seek */ pcmoffset = _initial_pcmoffset(vf,&vi); @@ -587,6 +584,7 @@ static int _bisect_forward_serialno(OggVorbis_File *vf, vf->pcmlengths[m*2+1]=searchgran; vf->pcmlengths[m*2+2]=pcmoffset; vf->pcmlengths[m*2+3]-=pcmoffset; + if(vf->pcmlengths[m*2+3]<0)vf->pcmlengths[m*2+3]=0; } return(0); @@ -634,11 +632,11 @@ static int _open_seekable2(OggVorbis_File *vf){ /* Get the offset of the last page of the physical bitstream, or, if we're lucky the last vorbis page of this link as most OggVorbis files will contain a single logical bitstream */ - end=_get_prev_page_serial(vf,vf->serialnos+2,vf->serialnos[1],&endserial,&endgran); + end=_get_prev_page_serial(vf,vf->end,vf->serialnos+2,vf->serialnos[1],&endserial,&endgran); if(end<0)return(end); /* now determine bitstream structure recursively */ - if(_bisect_forward_serialno(vf,0,dataoffset,vf->offset,endgran,endserial, + if(_bisect_forward_serialno(vf,0,dataoffset,end,endgran,endserial, vf->serialnos+2,vf->serialnos[1],0)<0)return(OV_EREAD); vf->offsets[0]=0; @@ -646,6 +644,7 @@ static int _open_seekable2(OggVorbis_File *vf){ vf->dataoffsets[0]=dataoffset; vf->pcmlengths[0]=pcmoffset; vf->pcmlengths[1]-=pcmoffset; + if(vf->pcmlengths[1]<0)vf->pcmlengths[1]=0; return(ov_raw_seek(vf,dataoffset)); } @@ -713,7 +712,7 @@ static int _fetch_and_process_packet(OggVorbis_File *vf, if(oldsamples)return(OV_EFAULT); vorbis_synthesis_blockin(&vf->vd,&vf->vb); - vf->samptrack+=vorbis_synthesis_pcmout(&vf->vd,NULL)-oldsamples; + vf->samptrack+=vorbis_synthesis_pcmout(&vf->vd,NULL); vf->bittrack+=op_ptr->bytes*8; } @@ -1401,22 +1400,41 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ if(pos>=total)break; } - /* search within the logical bitstream for the page with the highest - pcm_pos preceding (or equal to) pos. There is a danger here; - missing pages or incorrect frame number information in the - bitstream could make our task impossible. Account for that (it - would be an error condition) */ + /* Search within the logical bitstream for the page with the highest + pcm_pos preceding pos. If we're looking for a position on the + first page, bisection will halt without finding our position as + it's before the first explicit granulepos fencepost. That case is + handled separately below. + + There is a danger here; missing pages or incorrect frame number + information in the bitstream could make our task impossible. + Account for that (it would be an error condition) */ + + /* new search algorithm originally by HB (Nicholas Vinen) */ - /* new search algorithm by HB (Nicholas Vinen) */ { ogg_int64_t end=vf->offsets[link+1]; - ogg_int64_t begin=vf->offsets[link]; + ogg_int64_t begin=vf->dataoffsets[link]; ogg_int64_t begintime = vf->pcmlengths[link*2]; ogg_int64_t endtime = vf->pcmlengths[link*2+1]+begintime; ogg_int64_t target=pos-total+begintime; - ogg_int64_t best=begin; + ogg_int64_t best=-1; + int got_page=0; ogg_page og; + + /* if we have only one page, there will be no bisection. Grab the page here */ + if(begin==end){ + result=_seek_helper(vf,begin); + if(result) goto seek_error; + + result=_get_next_page(vf,&og,1); + if(result<0) goto seek_error; + + got_page=1; + } + + /* bisection loop */ while(beginoffset){ - result=_seek_helper(vf,bisect); - if(result) goto seek_error; - } + result=_seek_helper(vf,bisect); + if(result) goto seek_error; + /* read loop within the bisection loop */ while(beginoffset); if(result==OV_EREAD) goto seek_error; if(result<0){ + /* there is no next page! */ if(bisect<=begin+1) - end=begin; /* found it */ + /* No bisection left to perform. We've either found the + best candidate already or failed. Exit loop. */ + end=begin; else{ + /* We tried to load a fraction of the last page; back up a + bit and try to get the whole last page */ if(bisect==0) goto seek_error; bisect-=CHUNKSIZE; + + /* don't repeat/loop on a read we've already performed */ if(bisect<=begin)bisect=begin+1; + + /* seek and continue bisection */ result=_seek_helper(vf,bisect); if(result) goto seek_error; } }else{ ogg_int64_t granulepos; + got_page=1; + /* got a page. analyze it */ + /* only consider pages from primary vorbis stream */ if(ogg_page_serialno(&og)!=vf->serialnos[link]) continue; + /* only consider pages with the granulepos set */ granulepos=ogg_page_granulepos(&og); if(granulepos==-1)continue; if(granuleposoffset; /* raw offset of next page */ begintime=granulepos; + /* if we're before our target but within a short distance, + don't bisect; read forward */ if(target-begintime>44100)break; - bisect=begin; /* *not* begin + 1 */ + + bisect=begin; /* *not* begin + 1 as above */ }else{ - if(bisect<=begin+1) - end=begin; /* found it */ - else{ - if(end==vf->offset){ /* we're pretty close - we'd be stuck in */ + + /* This is one of our pages, but the granpos is + post-target; it is not a bisection return + candidate. (The only way we'd use it is if it's the + first page in the stream; we handle that case later + outside the bisection) */ + if(bisect<=begin+1){ + /* No bisection left to perform. We've either found the + best candidate already or failed. Exit loop. */ + end=begin; + }else{ + if(end==vf->offset){ + /* bisection read to the end; use the known page + boundary (result) to update bisection, back up a + little bit, and try again */ end=result; - bisect-=CHUNKSIZE; /* an endless loop otherwise. */ + bisect-=CHUNKSIZE; if(bisect<=begin)bisect=begin+1; result=_seek_helper(vf,bisect); if(result) goto seek_error; }else{ + /* Normal bisection */ end=bisect; endtime=granulepos; break; @@ -1486,9 +1533,46 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ } } - /* found our page. seek to it, update pcm offset. Easier case than - raw_seek, don't keep packets preceding granulepos. */ - { + /* Out of bisection: did it 'fail?' */ + if(best == -1){ + + /* Check the 'looking for data in first page' special case; + bisection would 'fail' because our search target was before the + first PCM granule position fencepost. */ + + if(got_page && + begin == vf->dataoffsets[link] && + ogg_page_serialno(&og)==vf->serialnos[link]){ + + /* Yes, this is the beginning-of-stream case. We already have + our page, right at the beginning of PCM data. Set state + and return. */ + + vf->pcm_offset=total; + + if(link!=vf->current_link){ + /* Different link; dump entire decode machine */ + _decode_clear(vf); + + vf->current_link=link; + vf->current_serialno=vf->serialnos[link]; + vf->ready_state=STREAMSET; + + }else{ + vorbis_synthesis_restart(&vf->vd); + } + + ogg_stream_reset_serialno(&vf->os,vf->current_serialno); + ogg_stream_pagein(&vf->os,&og); + + }else + goto seek_error; + + }else{ + + /* Bisection found our page. seek to it, update pcm offset. Easier case than + raw_seek, don't keep packets preceding granulepos. */ + ogg_page og; ogg_packet op; @@ -1518,23 +1602,23 @@ int ov_pcm_seek_page(OggVorbis_File *vf,ogg_int64_t pos){ while(1){ result=ogg_stream_packetpeek(&vf->os,&op); if(result==0){ - /* !!! the packet finishing this page originated on a - preceding page. Keep fetching previous pages until we - get one with a granulepos or without the 'continued' flag - set. Then just use raw_seek for simplicity. */ - - result=_seek_helper(vf,best); - if(result<0) goto seek_error; - - while(1){ - result=_get_prev_page(vf,&og); + /* No packet returned; we exited the bisection with 'best' + pointing to a page with a granule position, so the packet + finishing this page ('best') originated on a preceding + page. Keep fetching previous pages until we get one with + a granulepos or without the 'continued' flag set. Then + just use raw_seek for simplicity. */ + /* Do not rewind past the beginning of link data; if we do, + it's either a bug or a broken stream */ + result=best; + while(result>vf->dataoffsets[link]){ + result=_get_prev_page(vf,result,&og); if(result<0) goto seek_error; if(ogg_page_serialno(&og)==vf->current_serialno && (ogg_page_granulepos(&og)>-1 || !ogg_page_continued(&og))){ return ov_raw_seek(vf,result); } - vf->offset=result; } } if(result<0){